Merge pull request #3 from blackopseng3/master

Bugfix and improvements

Author: eric-martin

lib/stackify/authorization/authorizable.rb

@@ -30,11 +30,11 @@ def authorized!
 
     def successfull_authorisation response
       Stackify::EnvDetails.instance.update_auth_info JSON.parse(response.body)
-      Stackify.internal_log :info, 'Authorisation is finished successfully.'
+      Stackify.internal_log :info, 'Authorization is finished successfully.'
     end
 
     def unsuccessfull_authorisation response, caller
-      Stackify.log_internal_error "Authorisation finally failed: #{response_string(response)}"
+      Stackify.log_internal_error "Authorization finally failed: #{response_string(response)}"
       Stackify.shutdown_all caller unless @@authorized
     end
 

lib/stackify/authorization/authorization_client.rb

@@ -22,7 +22,7 @@ def auth_task attempts
         end
       }
       Stackify::ScheduleTask.new properties do
-        Stackify.internal_log :debug, 'AthorizationClient: trying to athorize...'
+        Stackify.internal_log :debug, 'AthorizationClient: trying to authorize...'
         send_request BASE_URI, Stackify::EnvDetails.instance.auth_info.to_json
       end
     end

lib/stackify/logger_client.rb

@@ -5,11 +5,11 @@ def initialize
       @@errors_governor = Stackify::ErrorsGovernor.new
     end
 
-    def log level, msg
+    def log level, msg, call_trace
       Stackify::Utils.do_only_if_authorized_and_mode_is_on Stackify::MODES[:logging] do
         if acceptable? level, msg && Stackify.working?
           worker = Stackify::AddMsgWorker.new
-          task = log_message_task level, msg
+          task = log_message_task level, msg, call_trace
           worker.async_perform ScheduleDelay.new, task
         end
       end
@@ -56,9 +56,12 @@ def is_appropriate_env?
       Stackify.configuration.env.downcase.to_sym == Stackify::EnvDetails.instance.auth_info['ConfiguredEnvironmentName'].downcase.to_sym
     end
 
-    def log_message_task level, msg
+    def log_message_task level, msg, call_trace
       Stackify::ScheduleTask.new ({limit: 1}) do
-        Stackify.msgs_queue << Stackify::MsgObject.new(level, msg, caller[0]).to_h
+        e = Exception.new(msg)
+        e.set_backtrace(call_trace)
+        ex = StackifiedError.new(e, binding())
+        Stackify.msgs_queue << Stackify::MsgObject.new(level, ex.message, caller[0], ex).to_h
       end
     end
 

lib/stackify/logger_proxy.rb

@@ -8,7 +8,7 @@ def initialize logger
         LoggerProxy.class_eval do
           define_method level.to_sym do |*args , &block|
             msg = message(args, block)
-            Stackify.logger_client.log(level.downcase, msg)
+            Stackify.logger_client.log(level.downcase, msg, caller)
             @logger.send(level.to_sym, args, &block)
           end
         end

lib/stackify/metrics/metric_aggregate.rb

@@ -38,7 +38,6 @@ def initialize aggr_metric
 
     def to_h
       {
-        'Count' => @count,
         'MetricType' => @metric_type,
         'MetricID' => @metric_id,
         'Name' => @name,

lib/stackify/metrics/metrics_client.rb

@@ -203,7 +203,7 @@ def upload_aggregates aggr_metrics
           mon_info = @monitor_ids[metric.name_key]
         else
           req = @metrics_sender.monitor_info metric
-          if req.try(:code) == '200'
+          if req.try(:status) == 200
             mon_info = JSON.parse req.body
             if !mon_info.nil? && !mon_info['MonitorID'].nil? && mon_info['MonitorID'] > 0
               @monitor_ids[metric.name_key] = mon_info
@@ -229,7 +229,7 @@ def upload_aggregates aggr_metrics
         #get identified once
         aggr_metrics_for_upload = aggr_metrics.select { |_key, aggr_metric| !aggr_metric.monitor_id.nil? }
         response = @metrics_sender.upload_metrics aggr_metrics_for_upload
-        Stackify.internal_log :info, 'Metrics are uploaded successfully' if response.try(:code) == '200'
+        Stackify.internal_log :info, 'Metrics are uploaded successfully' if response.try(:status) == 200
         all_success
       end
     end

lib/stackify/metrics/metrics_sender.rb

@@ -7,7 +7,7 @@ def monitor_info aggr_metric
       if Stackify.authorized?
         send_request GET_METRIC_INFO_URI, GetMetricRequest.new(aggr_metric).to_h.to_json
       else
-        Stackify.log_internal_error "Getting of monitor_info is failed because of authorisation failure"
+        Stackify.log_internal_error "Getting of monitor_info is failed because of authorization failure"
       end
     end
 
@@ -22,7 +22,7 @@ def upload_metrics aggr_metrics
         end
         send_request SUBMIT_METRIS_URI, records.to_json
       else
-        Stackify.log_internal_error "Uploading of metrics is failed because of authorisation failure"
+        Stackify.log_internal_error "Uploading of metrics is failed because of authorization failure"
       end
     end
   end