redhat_access 2.0.8 → 2.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 02277fe82af9a4a00bb4e2dc3f874e1ce2ed77b0
-  data.tar.gz: 5abf5704332d1241be651442f6f2493e67dc45e6
+  metadata.gz: 891c0c523197bcb2b164955aea3caebfc6a3f609
+  data.tar.gz: 1a69a775d400a364bd136a34cba866f067dbc8db
 SHA512:
-  metadata.gz: 6e92912d8c5568dca766f7cf22ec45dccab90c034e015f2bc18b5be89babf306ffa92f9bed23fd10c48caae619f89ed359fe075bb6073147910c3633923d0dba
-  data.tar.gz: 7c37ff2eeec95c251297ba0e15792e214d1fa8a1311b2016c05f187c056e140161ceb8cbe4864ca6a338fa8e1d71eca6d0a4d39eab42ed2532c72a2b25bc47c3
+  metadata.gz: 83df1989746684191b483b8ff07a5c12a2affb5a303670c33e9502cc8fdfeff321c40230917917ad622bf214c97ab03ad047aa9d6d83dedca2bc1bfefdea6592
+  data.tar.gz: 5023ca820fa8fdb563923cdeed710d5f3873d28a4b613c342e776cff42eb698596eba4ec7bf93d0736a22540849163f78dc536da3f4b90556d71230f1f3d3019

data/app/services/redhat_access/telemetry/messaging_service.rb

@@ -27,8 +27,8 @@ module RedhatAccess
                                  :high_percent => percent(rule_counts.error,rule_counts.total),
                                  :critical_percent => percent(rule_counts.critical,rule_counts.total)
                                 })
-        rescue
-          return error_response("Unable to get risk summary.")
+        rescue Exception => e
+          return error_response("Unable to get risk summary : #{e}")
         end
         # return {:critical_count => 10,
         #         :system_count => 100,
@@ -42,8 +42,8 @@ module RedhatAccess
       def rules_stat_summary
         begin
           http_get_from_json(RULE_STATS_PATH, {}, true)
-        rescue
-          return error_response("Unable to get rule summary.")
+        rescue Exception => e
+          return error_response("Unable to get rule summary : #{e}")
         end
       end
 
@@ -56,7 +56,7 @@ module RedhatAccess
             user_data = weekly_summary_data(user.login)
             data.push(user_data) unless user_data.nil? || user_data[:data].nil? || user_data[:data].total_systems == 0
           rescue => e
-            Rails.logger.warn("Unable to get weekly email data for user")
+            Rails.logger.warn("Unable to get weekly email data for user : #{e}")
           end
         end
         data
@@ -98,7 +98,7 @@ module RedhatAccess
         if machines.empty?
           machines = ['NULL_SET']
         end
-        machines
+        machines.sort
       end
 
       def get_current_organization
@@ -167,11 +167,18 @@ module RedhatAccess
                                                         options[:payload],
                                                         nil,
                                                         options[:use_subsets])
-        resp_data = res[:data]
-        if res[:code] != 200
-            raise "Unable to read data code #{res[:code]}"
+        if res.key?(:error)
+          raise res[:error]
+        end
+        res[:data]
+
+      end
+
+      def handle_errors(http_code)
+        return if http_code == 200 or http_code == 201
+        case http_code
+          when 401
         end
-        resp_data
       end
 
       def new_api_client(add_user_header)

data/app/views/dashboard/_insights_issues_widget.html.erb

@@ -15,7 +15,8 @@
     <% api = RedhatAccess::Telemetry::MessagingService.new(Organization.current) %>
     <% data = api.rules_stat_summary %>
     <% if data.query_error %>
-        <p class="ca"><%= _("There was an error retrieving Insights data.") %></p>
+        <p class="ca"><%= _("There was an error retrieving Insights data:") %></p>
+        <p class="ca"><%= data.query_error %></p>
     <% else %>
         <table class="table table-fixed table-striped table-bordered">
           <thead>

data/app/views/dashboard/_insights_risk_widget.html.erb

@@ -15,7 +15,8 @@
     <% api = RedhatAccess::Telemetry::MessagingService.new(Organization.current) %>
     <% risk_data = api.risk_summary %>
     <% if risk_data.query_error %>
-        <p class="ca"><%= _("There was an error retrieving Insights data.") %></p>
+        <p class="ca"><%= _("There was an error retrieving Insights data:") %></p>
+        <p class="ca"><%= risk_data.query_error %></p>
     <% else %>
         <table class="table table-fixed table-striped table-bordered">
           <thead>

data/lib/redhat_access/version.rb

@@ -1,3 +1,3 @@
 module RedhatAccess
-  VERSION = "2.0.8"
+  VERSION = "2.0.9"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redhat_access
 version: !ruby/object:Gem::Version
-  version: 2.0.8
+  version: 2.0.9
 platform: ruby
 authors:
 - Lindani Phiri
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-05 00:00:00.000000000 Z
+date: 2017-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redhat_access_lib