ougai 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 457d3322926c875ce7c6b72b7ae0f192a96788fe
-  data.tar.gz: eb2e3121f6a0b6367d7608b72344ca27ecb764ed
+  metadata.gz: 0235a013e7e1d174d71adb866072ce73243cb8a8
+  data.tar.gz: ffde73e617129cf7287abfd53cbd5f151e166816
 SHA512:
-  metadata.gz: eb7ebaf9a4b3bd092410aef5778152f378e53a2a059cb450170e536bea78c6a64919be9325cc285e6b9978dfbf51d551e21155d544c492f9ea58904652acb5e6
-  data.tar.gz: 0f0d12daf3ad7959e8fb7c65599d07af6b427bdd8be262904b87781d6a439a9ba453a738387a496ee3bf677471609de9e6ebe64f7ea125dd957d1964c150a649
+  metadata.gz: be64d5eb15dca49f85a323b2c3fac463d83a42dd3f5f0bca6d318ca27cd8646e74dd3e3d658d3248469ccf4e6320c39512a390a91ca92d8a46dde3ab1f7f4c3a
+  data.tar.gz: a634e9fa73595c01748390bb6043e220c8f0b65cc052202320cf6fa31ea4b3be0984c623171afc46ea72e227d3f32bceb770b3baf07d8f0421e54067ac2326f8

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ougai (1.2.0)
+    ougai (1.3.0)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -214,6 +214,28 @@ gc_logger.debug('something detail', age: 34, weight: 72)
 
 If any field exists in both parent log and child log, the parent value is overridden or merged by child value.
 
+### Hook before logging
+
+Setting `before_log` of logger or child an *lambda* with `data` field, a process can be run before log each output.
+
+* Adding variable data (like Thread ID) to logging data can be defined in common.
+* Returning `false` in *lambda*, the log is cancelled and does not output.
+* The *before_log* of child logger is run ahead of the parent logger's.
+
+```ruby
+logger.before_log = lambda do |data|
+  data[:thread_id] = Thread.current.object_id.to_s(36)
+end
+
+logger.debug('on main thread')
+Thread.new { logger.debug('on another thread') }
+```
+
+```json
+{"name":"main","hostname":"mint","pid":13975,"level":20,"time":"2017-08-06T15:35:53.435+09:00","v":0,"msg":"on main thread","thread_id":"gqe0ava6c"}
+{"name":"main","hostname":"mint","pid":13975,"level":20,"time":"2017-08-06T15:35:53.435+09:00","v":0,"msg":"on another thread","thread_id":"gqe0cb14g"}
+```
+
 ## View log by node-bunyan
 
 Install [bunyan](https://github.com/trentm/node-bunyan) via NPM

data/lib/ougai/child_logger.rb

@@ -11,14 +11,16 @@ module Ougai
       @parent.level
     end
 
-    def chain(severity, args, fields)
-      @parent.chain(severity, args, merge_fields(@with_fields, fields))
+    def chain(severity, args, fields, hooks)
+      hooks.push(@before_log) if @before_log
+      @parent.chain(severity, args, merge_fields(@with_fields, fields), hooks)
     end
 
     protected
 
     def append(severity, args)
-      @parent.chain(severity, args, @with_fields)
+      hooks = @before_log ? [@before_log] : []
+      @parent.chain(severity, args, @with_fields, hooks)
     end
   end
 end

data/lib/ougai/logger.rb

@@ -25,22 +25,28 @@ module Ougai
       end
     end
 
-    def chain(severity, args, fields)
-      write(severity, args, merge_fields(@with_fields, fields))
+    def chain(severity, args, fields, hooks)
+      hooks.push(@before_log) if @before_log
+      write(severity, args, merge_fields(@with_fields, fields), hooks)
     end
 
     protected
 
     def append(severity, args)
-      write(severity, args, @with_fields)
+      hooks = @before_log ? [@before_log] : []
+      write(severity, args, @with_fields, hooks)
     end
 
     def create_formatter
       Formatters::Bunyan.new
     end
 
-    def write(severity, args, fields)
-      add(severity, merge_fields(fields, to_item(args)))
+    def write(severity, args, fields, hooks)
+      data = merge_fields(fields, to_item(args))
+      hooks.each do |hook|
+        return false if hook.call(data) == false
+      end
+      add(severity, data)
     end
 
     def to_item(args)

data/lib/ougai/logging.rb

@@ -1,6 +1,7 @@
 module Ougai
   module Logging
     attr_accessor :with_fields
+    attr_writer :before_log
 
     def debug(message = nil, ex = nil, data = nil, &block)
       return true if level > Logger::DEBUG
@@ -41,7 +42,7 @@ module Ougai
       ChildLogger.new(self, fields)
     end
 
-    def chain(severity, args, fields)
+    def chain(_severity, _args, _fields, _hooks)
       raise NotImplementedError
     end
 

data/lib/ougai/version.rb

@@ -1,3 +1,3 @@
 module Ougai
-  VERSION = "1.2.0"
+  VERSION = "1.3.0"
 end

data/spec/child_logger_spec.rb

@@ -297,4 +297,60 @@ describe Ougai::ChildLogger do
       end
     end
   end
+
+  describe '#before_log' do
+    let(:logger) { parent_logger.child }
+    let(:log_msg) { 'before_log test' }
+
+    before do
+      parent_logger.level = Logger::INFO
+    end
+
+    context 'child logger to be set before_log' do
+      before do
+        logger.before_log = lambda do |data|
+          data[:context_id] = 123
+        end
+      end
+
+      it 'outputs the field to be added in before_log' do
+        logger.info(log_msg)
+        expect(item).to be_log_message(log_msg, 30)
+        expect(item).to include(context_id: 123)
+      end
+    end
+
+    context 'parent logger to be set before_log' do
+      before do
+        parent_logger.before_log = lambda do |data|
+          data[:context_id] = 12345
+        end
+      end
+
+      it 'outputs the field to be added in before_log' do
+        logger.info(log_msg)
+        expect(item).to be_log_message(log_msg, 30)
+        expect(item).to include(context_id: 12345)
+      end
+    end
+
+    context 'both child logger and parent logger to be set before_log' do
+      before do
+        logger.before_log = lambda do |data|
+          data[:context_id] = 67890
+          data[:context_name] = 'sub'
+        end
+        parent_logger.before_log = lambda do |data|
+          data[:context_id] = 12345
+        end
+      end
+
+      it 'outputs the fields to be added in each before_log' do
+        logger.info(log_msg)
+        expect(item).to be_log_message(log_msg, 30)
+        expect(item).to include(context_id: 12345) # parent
+        expect(item).to include(context_name: 'sub') # child
+      end
+    end
+  end
 end

data/spec/logger_spec.rb

@@ -483,4 +483,37 @@ describe Ougai::Logger do
       end
     end
   end
+
+  describe '#before_log' do
+    let(:log_msg) { 'before_log test' }
+
+    context 'set context data' do
+      before do
+        logger.level = Logger::INFO
+        logger.before_log = lambda do |data|
+          data[:context_id] = 123
+        end
+      end
+
+      it 'outputs with context data' do
+        logger.info(log_msg)
+        expect(item).to be_log_message(log_msg, 30)
+        expect(item).to include(context_id: 123)
+      end
+    end
+
+    context 'cancelling log' do
+      before do
+        logger.level = Logger::INFO
+        logger.before_log = lambda do |data|
+          false
+        end
+      end
+
+      it 'outputs none' do
+        logger.info(log_msg)
+        expect(item).to be_nil
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ougai
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Toshimitsu Takahashi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-01 00:00:00.000000000 Z
+date: 2017-08-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler