chili_logger 0.0.6 → 0.0.10

data/lib/{logs_coverage → helpers/logs_coverage}/coverage_writer.rb

@@ -1,16 +1,18 @@
 require 'yaml'
 require 'fileutils'
+require 'helpers/values/default'
 
 # class for centralizing Creative's logging logic
 class ChiliLogger
   # class for keeping logs coverage up to date
   class CoverageWriter
     def self.write(desc, backtrace)
+      default = ChiliLogger::Values::Default.new
       coverage = read_file
 
-      type = desc[:type]
-      service = desc[:service]
-      action = desc[:action]
+      type = desc[:type] || default.type
+      service = desc[:service] || default.service
+      action = desc[:action] || default.action
       keys = [type, service, action]
 
       coverage = add_nested_value(coverage, keys, backtrace)
@@ -18,17 +20,6 @@ class ChiliLogger
       write_file(coverage)
     end
 
-    def self.add_nested_value(object, keys, final_value)
-      return final_value if keys.empty?
-
-      current_key = keys.first
-      nested_object = object[current_key] || {}
-      nested_keys = keys.drop(1)
-      object[current_key] = add_nested_value(nested_object, nested_keys, final_value)
-
-      object
-    end
-
     def self.read_file
       file_exists = File.file?(file_path)
 
@@ -41,6 +32,19 @@ class ChiliLogger
       true
     end
 
+    # private class methods
+
+    def self.add_nested_value(object, keys, final_value)
+      return final_value if keys.empty?
+
+      current_key = keys.first
+      nested_object = object[current_key] || {}
+      nested_keys = keys.drop(1)
+      object[current_key] = add_nested_value(nested_object, nested_keys, final_value)
+
+      object
+    end
+
     def self.dir_name
       'log'
     end
@@ -52,5 +56,7 @@ class ChiliLogger
     def self.file_path
       File.join('.', dir_name, file_name)
     end
+
+    private_class_method :add_nested_value, :dir_name, :file_name, :file_path
   end
 end

data/lib/helpers/values/default.rb

@@ -0,0 +1,92 @@
+# class for centralizing Creative's logging logic
+class ChiliLogger
+  # module responsible for uniformization of values in ChiliLogger 
+  module Values
+    # class for keeping all default values in a single place
+    class Default
+      def msg_broker
+        :rabbitmq
+      end
+
+      def msg_broker_config
+        {}
+      end
+
+      def user
+        {
+          cognito_id: undefined_string,
+          email: undefined_string,
+          company_cognito_id: undefined_string,
+          company_name: undefined_string,
+          ghost_user_cognito_id: undefined_string
+        }
+      end
+
+      def env
+        undefined_string
+      end
+
+      def layer
+        undefined_string
+      end
+
+      def type
+        undefined_string
+      end
+
+      def service
+        undefined_string
+      end
+
+      def action
+        undefined_string
+      end
+
+      def desc
+        { type: type, service: service, action: action }
+      end
+
+      def main_content
+        { modified_records: modified_records, errors: log_errors }
+      end
+
+      def modified_records
+        {}
+      end
+
+      def log_errors
+        []
+      end
+
+      def log_error
+        sentence1 = 'ChiliLogger received an unsupported type of error, so it replaced it with this generic string.'
+        sentence2 = ' Errors must be either a String or a descendant of the Exception class.'
+        sentence1 + sentence2
+      end
+
+      def server_url
+        undefined_string
+      end
+
+      def log
+        {}
+      end
+
+      def sqs_message
+        {
+          error_type: undefined_string,
+          error_message: undefined_string,
+          log: undefined_string
+        }
+      end
+
+      def invalid_string
+        'invalid_format'
+      end
+
+      def undefined_string
+        'not_specified'
+      end
+    end
+  end
+end

data/lib/helpers/values/type_uniformizer/desc.rb

@@ -0,0 +1,64 @@
+# class for centralizing Creative's logging logic
+class ChiliLogger
+  # module responsible for uniformization of values in ChiliLogger
+  module Values
+    # class to enforce consistent typing of values across all classes
+    # this is important because of Elasticsearch. Check README, section ##read-before-contributing to know more about it
+    class TypeUniformizer
+      # responsible for uniformizing log's description primitive types
+      class Desc
+        def initialize
+          @default = ChiliLogger::Values::Default.new
+        end
+
+        def desc(desc)
+          return {} unless desc.is_a?(Hash)
+
+          uniformized_desc = {}
+          valid_desc_attrs.each do |attr_name|
+            # only overwrite if key was explicitly set in desc
+            next unless desc.key?(attr_name)
+
+            desc_attr = desc[attr_name]
+            desc_attr = @default.invalid_string unless valid_desc_attr_formats.include?(desc_attr.class.name)
+            desc_attr ||= @default.undefined_string # if type == nil
+            uniformized_desc[attr_name] = desc_attr
+          end
+
+          uniformized_desc
+        end
+
+        def type(type)
+          return @default.undefined_string if type.nil?
+          return @default.invalid_string unless type.is_a?(String)
+
+          type
+        end
+
+        def service(service)
+          return @default.undefined_string if service.nil?
+          return @default.invalid_string unless service.is_a?(String)
+
+          service
+        end
+
+        def action(action)
+          return @default.undefined_string if action.nil?
+          return @default.invalid_string unless action.is_a?(String)
+
+          action
+        end
+
+        private
+
+        def valid_desc_attrs
+          %i[type service action]
+        end
+
+        def valid_desc_attr_formats
+          %w[String NilClass]
+        end
+      end
+    end
+  end
+end

data/lib/helpers/values/type_uniformizer/main_content.rb

@@ -0,0 +1,62 @@
+# class for centralizing Creative's logging logic
+class ChiliLogger
+  # module responsible for uniformization of values in ChiliLogger
+  module Values
+    # class to enforce consistent typing of values across all classes, this is important because of Elasticsearch
+    # Check README, section ##read-before-contributing, to know more about it
+    class TypeUniformizer
+      # responsible for uniformizing the primitive types of a log's main_content
+      class MainContent
+        def initialize
+          @default = ChiliLogger::Values::Default.new
+        end
+
+        def main_content(content)
+          return {} unless content.is_a?(Hash)
+
+          uniformized_content = {}
+
+          if content.key?(:modified_records) # only overwrite if :modified_records was explicitly set in main_content
+            mod_recs = content[:modified_records]
+            uniformized_content[:modified_records] = modified_records(mod_recs)
+          end
+          # only overwrite if :errors was explicitly set in main_content
+          uniformized_content[:errors] = errors(content[:errors]) if content.key?(:errors)
+ 
+          uniformized_content
+        end
+
+        def modified_records(records)
+          return {} unless records.is_a?(Hash)
+
+          uniformized_recs = {}
+          # modified records should be a hash where each key points to an array of valid modified records
+          records.each do |key, val|
+            val_as_array = val.is_a?(Array) ? val : [].push(val)
+            uniformized_val = val_as_array.map { |record| modified_record(record) }
+            uniformized_recs[key.to_s] = uniformized_val
+          end
+
+          uniformized_recs
+        end
+
+        def modified_record(record)
+          record.is_a?(Hash) ? record : {}
+        end
+
+        def errors(errors)
+          return [] unless errors.is_a?(Array)
+
+          errors.map { |err| error(err) }
+        end
+
+        # a received error must be either a string or Exception descendent(it will be converted to a string)
+        def error(error)
+          error = error.inspect if error.class.ancestors.include?(Exception) # stringifies Exception descendents
+          error = @default.log_error unless error.is_a?(String)
+          error
+        end
+      end
+    end
+  end
+end

data/lib/helpers/values/type_uniformizer/user.rb

@@ -0,0 +1,43 @@
+# class for centralizing Creative's logging logic
+class ChiliLogger
+  # module responsible for uniformization of values in ChiliLogger
+  module Values
+    # class to enforce consistent typing of values across all classes
+    # this is important because of Elasticsearch. Check README, section ##read-before-contributing to know moreabout it
+    class TypeUniformizer
+      # responsible for uniformizing log's primitive types
+      class User
+        def initialize
+          @default = ChiliLogger::Values::Default.new
+        end
+
+        def user(user)
+          return {} unless user.is_a?(Hash)
+
+          uniformized_user = {}
+          valid_user_attrs.each do |attr_name|
+            # don't enforce typing compliance of attrs that weren't explicitly set in user
+            next unless user.key?(attr_name)
+
+            user_attr = user[attr_name]
+            user_attr = @default.invalid_string unless valid_user_attr_formats.include?(user_attr.class.name)
+            user_attr ||= @default.undefined_string # if type == nil
+            uniformized_user[attr_name] = user_attr.to_s
+          end
+
+          uniformized_user
+        end
+
+        private
+
+        def valid_user_attrs
+          %i[cognito_id email company_cognito_id company_name ghost_user_cognito_id]
+        end
+
+        def valid_user_attr_formats
+          %w[String NilClass Fixnum Bignum]
+        end
+      end
+    end
+  end
+end

data/lib/message_writer/aws_ops_metadata.rb

@@ -1,5 +1,5 @@
 require 'httparty'
-require 'logs_coverage/coverage_writer'
+require 'helpers/logs_coverage/coverage_writer'
 require 'message_writer/aws_ops_metadata'
 
 # class for centralizing Creative's logging logic

data/lib/message_writer/message_writer.rb

@@ -1,4 +1,4 @@
-require 'logs_coverage/coverage_writer'
+require 'helpers/logs_coverage/coverage_writer'
 require 'message_writer/aws_ops_metadata'
 
 # class for centralizing Creative's logging logic
@@ -6,21 +6,23 @@ class ChiliLogger
   # class for writing log messages
   class MessageWriter
     def initialize(config)
-      @env = config[:env] || 'not_specified'
-      @layer = config[:layer] || 'not_specified'
-      @server_url = config[:server_url] || 'not_specified'
+      @default = ChiliLogger::Values::Default.new
+      @env = config[:env] || @default.env
+      @layer = config[:layer] || @default.layer
+      @server_url = config[:server_url] || @default.server_url
       @cloud_provider = config[:cloud_provider]
     end
 
-    def write(desc = {}, agent = 'not_specified', main_content = {})
+    def write(**options)
       return if ChiliLogger.instance.deactivated
 
-      desc ||= {}
-      main_content ||= {}
+      desc = options[:desc] || @default.desc
+      user = options[:user] || @default.user
+      main_content = options[:main_content] || @default.main_content
 
-      main_content.merge!( backtrace: clean_backtrace(caller) )
+      main_content[:backtrace] = clean_backtrace(caller)
       update_logs_coverage(desc, main_content)
-      message_hash(desc, agent, main_content)
+      message_hash(desc: desc, user: user, main_content: main_content)
     end
 
     def clean_backtrace(backtrace)
@@ -31,9 +33,9 @@ class ChiliLogger
     def write_description_tag(desc)
       env = desc[:env] || @env
       layer = desc[:layer] || @layer
-      type = desc[:type] || 'not_specified'
-      service = desc[:service] || 'not_specified'
-      action = desc[:action] || 'not_specified'
+      type = desc[:type] || @default.type
+      service = desc[:service] || @default.service
+      action = desc[:action] || @default.action
 
       "#{env}.#{layer}.#{type}.#{service}.#{action}"
     end
@@ -47,17 +49,18 @@ class ChiliLogger
     end
 
     # rubocop:disable Metrics/MethodLength
-    def message_hash(desc, agent, main_content)
+    def message_hash(**options)
+      desc = options[:desc]
       {
         desc: write_description_tag(desc),
         env: @env,
         layer: @layer,
-        type: desc[:type] || 'not_specified',
-        service: desc[:service] || 'not_specified',
-        action: desc[:action] || 'not_specified',
-        user_info: agent || 'not_specified',
+        type: desc[:type] || @default.type,
+        service: desc[:service] || @default.service,
+        action: desc[:action] || @default.action,
+        user: options[:user] || @default.undefined_string,
         timestamp: DateTime.now,
-        "#{desc[:type]}": main_content || {},
+        "#{desc[:type]}": options[:main_content] || {},
         ops_metadata: write_ops_metadata
       }
     end

data/log/chili-logger-coverage.yml

@@ -2,7 +2,7 @@
 test:
   console:
     publish_test:
-    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/chili_logger-0.0.2/lib/chili_logger.rb:42:in
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/chili_logger-0.0.8/lib/chili_logger.rb:61:in
       `publish_instant_log'"
     - "(pry):28:in `__pry__'"
     - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:290:in
@@ -48,3 +48,52 @@ test:
     - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb/init.rb:20:in `setup'"
     - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb.rb:378:in `start'"
     - "/usr/share/rvm/rubies/ruby-2.3.3/bin/irb:11:in `<main>'"
+transaction:
+  console:
+    publish_test:
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/chili_logger-0.0.8/lib/chili_logger.rb:61:in
+      `publish_instant_log'"
+    - "(pry):53:in `__pry__'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:290:in
+      `eval'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:290:in
+      `evaluate_ruby'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:659:in
+      `handle_line'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:261:in
+      `block (2 levels) in eval'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:260:in
+      `catch'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:260:in
+      `block in eval'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:259:in
+      `catch'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_instance.rb:259:in
+      `eval'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:77:in `block
+      in repl'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:67:in `loop'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:67:in `repl'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:38:in `block
+      in start'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/input_lock.rb:61:in
+      `__with_ownership'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/input_lock.rb:78:in
+      `with_ownership'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:38:in `start'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/repl.rb:15:in `start'"
+    - "/home/lucas/.rvm/gems/ruby-2.3.3/gems/pry-0.13.1/lib/pry/pry_class.rb:191:in
+      `start'"
+    - "/home/lucas/.irbrc:8:in `<top (required)>'"
+    - "/usr/share/rvm/scripts/irbrc.rb:40:in `load'"
+    - "/usr/share/rvm/scripts/irbrc.rb:40:in `<top (required)>'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:54:in
+      `require'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:54:in
+      `require'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/.irbrc:11:in `<top (required)>'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb/init.rb:231:in `load'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb/init.rb:231:in `run_config'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb/init.rb:20:in `setup'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/lib/ruby/2.3.0/irb.rb:378:in `start'"
+    - "/usr/share/rvm/rubies/ruby-2.3.3/bin/irb:11:in `<main>'"