epilog 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c73f8fcac8b47000824334dce31d8b781aa5e730201f2e67816c4c395cbe3966
-  data.tar.gz: 7f45a7801cb9a815cb576c39de3fb49c09f6b928e7cc2f2b1b50435df548d0e1
+  metadata.gz: ab9c7483e1069a23f488152643f2bfb8bd2598956314b8735ad220acec8c4472
+  data.tar.gz: 80e15c598d75805161137278ec98ed3a134526e006a9c26a7e6f1dcbb2c4025d
 SHA512:
-  metadata.gz: ff73c27fe2100ffc880d689196fc4deed995c3e2978f4d01412bdd63ec7b73de3e11903af43570a0ca29a5040fec275d6a4fad0cbfdf2d0d41bd01da11469045
-  data.tar.gz: 34bb7195d88534537637e43a729314ba6439f3e40a4c150c33c46c0bb0aa41e9bf544e0f0d47e10cff534c7e71877b53206b5f928c412e9337da471846a34b75
+  metadata.gz: 107b41212249b6abf84deb5a13460f4cfe3ee7e510c311b4bc1907528947fc85ef5fa35d030c2e79607583584ecbc6d4466d3b91ff12992c3fc202e55a5a49ab
+  data.tar.gz: b2c35471c0b4241faeba4551716f9c4393ccdf717ff411c7d6f4f414e4a6949d0f2a3aa93347ddaf244ea4096eea3ba4cf04b6dd47867a8eb50a3aa48c6bbc66

data/.rubocop.yml

@@ -2,24 +2,30 @@
 AllCops:
   TargetRubyVersion: 2.3
 
+Layout/AlignArguments:
+  EnforcedStyle: with_fixed_indentation
+
 Layout/EndAlignment:
   EnforcedStyleAlignWith: variable
 
 Layout/AlignParameters:
   EnforcedStyle: with_fixed_indentation
 
-Layout/IndentArray:
+Layout/IndentFirstArgument:
   EnforcedStyle: consistent
 
-Layout/IndentHash:
+Layout/IndentFirstArrayElement:
   EnforcedStyle: consistent
 
-Style/Documentation:
-  Enabled: false
+Layout/IndentFirstHashElement:
+  EnforcedStyle: consistent
 
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented
 
+Style/Documentation:
+  Enabled: false
+
 Style/FrozenStringLiteralComment:
   Enabled: true
   EnforcedStyle: always

data/CHANGLOG.md

@@ -0,0 +1,14 @@
+# Changelog
+
+## 0.3
+
+- Update development dependencies [#1](https://github.com/machinima/epilog/pull/1)
+- Allow controllers to add context to logs [#2](https://github.com/machinima/epilog/pull/2)
+- Fix removing Rails default parameters when filter_parameters are set [#4](https://github.com/machinima/epilog/pull/4)
+- Add a filter using Rails filter_parameters [#3](https://github.com/machinima/epilog/pull/3)
+- Change action view logs to DEBUG [#5](https://github.com/machinima/epilog/pull/5)
+- Filter all request data using filter_parameters [#6](https://github.com/machinima/epilog/pull/6)
+
+## 0.2
+
+Initial public release

data/epilog.gemspec

@@ -19,16 +19,17 @@ Gem::Specification.new do |spec|
     .reject { |f| f.match(%r{^spec/}) }
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'bundler', '~> 1.12'
-  spec.add_development_dependency 'byebug', '~> 9.0'
-  spec.add_development_dependency 'combustion', '~> 1.0.0'
+  spec.add_development_dependency 'bundler', '>= 1.12'
+  spec.add_development_dependency 'byebug', '~> 11.0'
+  spec.add_development_dependency 'combustion', '~> 1.1.0'
+  spec.add_development_dependency 'irb'
   spec.add_development_dependency 'rails', '>= 4.2', '< 6'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'redcarpet', '~> 3.4'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'redcarpet', '~> 3.5'
   spec.add_development_dependency 'rspec', '~> 3.4'
   spec.add_development_dependency 'rspec-rails', '~> 3.8.1'
-  spec.add_development_dependency 'rubocop', '~> 0.61'
-  spec.add_development_dependency 'simplecov', '~> 0.12'
-  spec.add_development_dependency 'sqlite3', '~> 1.3'
+  spec.add_development_dependency 'rubocop', '0.75'
+  spec.add_development_dependency 'simplecov', '~> 0.17'
+  spec.add_development_dependency 'sqlite3', '~> 1.3', '< 1.4'
   spec.add_development_dependency 'yard', '~> 0.9.11'
 end

data/lib/epilog/filter.rb

@@ -2,3 +2,4 @@
 
 require 'epilog/filter/hash_key'
 require 'epilog/filter/blacklist'
+require 'epilog/filter/filter_parameters'

data/lib/epilog/filter/filter_parameters.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Epilog
+  module Filter
+    class FilterParameters < Blacklist
+      private
+
+      def filter_parameters
+        return @filter_parameters if @filter_parameters
+
+        filtered = Hash[
+          ::Rails.application.config.filter_parameters.map do |p|
+            [p.to_s.downcase, true]
+          end
+        ]
+
+        @filter_parameters = filtered if ::Rails.initialized?
+        filtered
+      end
+
+      def key?(key)
+        filter_parameters.key?(key.to_s.downcase)
+      end
+    end
+  end
+end

data/lib/epilog/rails/action_controller_subscriber.rb

@@ -17,13 +17,13 @@ module Epilog
 
       def process_request(event)
         info do
-          {
+          event.payload[:context].merge(
             message: response_string(event),
             request: short_request_hash(event),
             response: response_hash(event),
             metrics: process_metrics(event.payload[:metrics]
               .merge(request_runtime: event.duration.round(2)))
-          }
+          )
         end
       end
 
@@ -79,6 +79,8 @@ module Epilog
 
       def request_hash(event) # rubocop:disable AbcSize, MethodLength
         request = event.payload[:request]
+        param_filter = request.send(:parameter_filter)
+
         {
           id: request.uuid,
           ip: request.remote_ip,
@@ -86,13 +88,15 @@ module Epilog
           protocol: request.protocol.to_s.gsub('://', ''),
           method: request.request_method,
           port: request.port,
-          path: request.fullpath,
-          query: request.query_parameters,
-          cookies: request.cookies,
-          headers: request.headers.to_h.keep_if do |key, _value|
-            key =~ ActionDispatch::Http::Headers::HTTP_HEADER
-          end,
-          params: request.filtered_parameters.except(*RAILS_PARAMS),
+          path: request.path,
+          query: param_filter.filter(request.query_parameters),
+          cookies: param_filter.filter(request.cookies),
+          headers: param_filter.filter(
+            request.headers.to_h.keep_if do |key, _value|
+              key =~ ActionDispatch::Http::Headers::HTTP_HEADER
+            end
+          ),
+          params: request.filtered_parameters.except(*rails_params),
           format: request.format.try(:ref),
           controller: event.payload[:controller],
           action: event.payload[:action]
@@ -104,13 +108,13 @@ module Epilog
         {
           id: request.uuid,
           method: request.method,
-          path: request.fullpath
+          path: request.path
         }
       end
 
       def request_string(event)
         request = event.payload[:request]
-        "#{request.request_method} #{request.fullpath}"
+        "#{request.request_method} #{request.path}"
       end
 
       def response_hash(event)
@@ -147,6 +151,10 @@ module Epilog
           obj[key] = value.round(2) if value.is_a?(Numeric)
         end
       end
+
+      def rails_params
+        @rails_params ||= RAILS_PARAMS + RAILS_PARAMS.map(&:to_s)
+      end
     end
   end
 end

data/lib/epilog/rails/action_view_subscriber.rb

@@ -4,15 +4,15 @@ module Epilog
   module Rails
     class ActionViewSubscriber < LogSubscriber
       def render_template(event)
-        info { hash(event, 'Rendered template') }
+        debug { hash(event, 'Rendered template') }
       end
 
       def render_partial(event)
-        info { hash(event, 'Rendered partial') }
+        debug { hash(event, 'Rendered partial') }
       end
 
       def render_collection(event)
-        info { hash(event, 'Rendered collection') }
+        debug { hash(event, 'Rendered collection') }
       end
 
       private

data/lib/epilog/rails/ext/action_controller.rb

@@ -10,6 +10,7 @@ module Epilog
         ensure
           payload[:response] = response
           payload[:metrics] = epilog_metrics
+          payload[:context] = epilog_context
         end
       end
     end
@@ -39,6 +40,10 @@ module Epilog
         view_runtime: view_runtime
       }
     end
+
+    def epilog_context
+      {}
+    end
   end
 end
 

data/lib/epilog/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Epilog
-  VERSION = '0.2.0'
+  VERSION = '0.3.0'
 end

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: epilog
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Justin Howard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-12-18 00:00:00.000000000 Z
+date: 2019-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.12'
 - !ruby/object:Gem::Dependency
@@ -30,28 +30,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.0'
+        version: '11.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.0'
+        version: '11.0'
 - !ruby/object:Gem::Dependency
   name: combustion
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
+- !ruby/object:Gem::Dependency
+  name: irb
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -78,28 +92,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '3.5'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -132,30 +146,30 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0.61'
+        version: '0.75'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0.61'
+        version: '0.75'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.17'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -163,6 +177,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '1.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -170,6 +187,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '1.4'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -196,6 +216,7 @@ files:
 - ".rubocop.yml"
 - ".travis.yml"
 - ".yardopts"
+- CHANGLOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -212,6 +233,7 @@ files:
 - lib/epilog.rb
 - lib/epilog/filter.rb
 - lib/epilog/filter/blacklist.rb
+- lib/epilog/filter/filter_parameters.rb
 - lib/epilog/filter/hash_key.rb
 - lib/epilog/log_formatter.rb
 - lib/epilog/logger.rb
@@ -251,7 +273,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.8
+rubygems_version: 2.7.7
 signing_key: 
 specification_version: 4
 summary: A JSON logger with Rails support