getaround_utils 0.2.11 → 0.2.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c5d5708a53cbea1e3d9ac5a8f955f953174c30f6e2346894e2da2cb02754086
-  data.tar.gz: c7b4896ff5bebb107944d1143bd01936cc89a1190c88c770d7b03fd6fd404988
+  metadata.gz: 51565d059ccaf6283cf55e699ccd205e4f043efd597c093739f45e52fdfba22c
+  data.tar.gz: 9f28c08c0f04120166c802c158dc8fa1263a3304e63c1e498d58f1df926987b5
 SHA512:
-  metadata.gz: ec6aad3ef2675a176404d6f64c8c66ec5cb0a2386546d3a9e5a03b0648884f144130a58186b81e57d2d3d6dee8fbb22cb0066fd2a201f216a60ace96716aae7e
-  data.tar.gz: b10937d3b045a8dc8c69df67a2db5658729734615afe90aa9602e7a77c253ef4aac14e552028b221bc9ceeba2ebd9e104d6c12c602d034c492f12737d8c73c7b
+  metadata.gz: 384fe8dbcbd3b6fbaea8b093a9217daeaaf3bd2c3432c7bb3bc4e7204055edf75e2b3f0af6926c17328c1f80b0ee054c05b7410f772734a99916cfe3eaf3ac4f
+  data.tar.gz: 9040d753022b812502f084dc3c3cef369f6b4ca720123443d95f7c17ee2c2683cdba97505a453c9e584596413f9ba2abd6f052ef6aa4c974e59ef0b779b8c1a3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    getaround_utils (0.2.11)
+    getaround_utils (0.2.13)
 
 GEM
   remote: https://rubygems.org/
@@ -103,7 +103,7 @@ GEM
     minitest (5.13.0)
     multipart-post (2.1.1)
     nio4r (2.5.2)
-    nokogiri (1.10.5)
+    nokogiri (1.10.10)
       mini_portile2 (~> 2.4.0)
     oj (3.10.14)
     ougai (1.8.5)
@@ -115,7 +115,7 @@ GEM
       coderay (~> 1.1.0)
       method_source (~> 0.9.0)
     public_suffix (4.0.2)
-    rack (2.0.7)
+    rack (2.2.3)
     rack-protection (2.0.7)
       rack
     rack-test (1.1.0)
@@ -147,7 +147,7 @@ GEM
       rake (>= 0.8.7)
       thor (>= 0.20.3, < 2.0)
     rainbow (3.0.0)
-    rake (10.5.0)
+    rake (12.3.3)
     redis (4.1.3)
     relaxed-rubocop (2.4)
     request_store (1.4.1)
@@ -209,7 +209,7 @@ GEM
       hashdiff (>= 0.4.0, < 2.0.0)
     websocket-driver (0.7.1)
       websocket-extensions (>= 0.1.0)
-    websocket-extensions (0.1.4)
+    websocket-extensions (0.1.5)
     zeitwerk (2.2.1)
 
 PLATFORMS
@@ -224,7 +224,7 @@ DEPENDENCIES
   ougai (~> 1.8)
   pry (~> 0.12.2)
   rails (~> 6.0)
-  rake (~> 10.0)
+  rake (~> 12.3)
   request_store_rails (~> 2.0)
   rspec (~> 3.9, >= 3.9.0)
   rspec-rails (~> 3.9)

data/getaround_utils.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'lograge', '~> 0.11.2'
   gem.add_development_dependency 'pry', '~> 0.12.2'
   gem.add_development_dependency 'rails', '~> 6.0'
-  gem.add_development_dependency "rake", "~> 10.0"
+  gem.add_development_dependency "rake", "~> 12.3"
   gem.add_development_dependency 'rspec', '~> 3.9', '>= 3.9.0'
   gem.add_development_dependency 'rspec-rails', '~> 3.9'
   gem.add_development_dependency 'rubocop', '~> 0.75', '>= 0.75.0'

data/lib/getaround_utils/railties/lograge.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails/railtie'
 require 'lograge'
 
@@ -8,13 +10,12 @@ class GetaroundUtils::Railties::Lograge < Rails::Railtie
   module LogrageActionController
     def append_info_to_payload(payload)
       super
-      payload[:lograge] ||= {}
+      payload[:lograge] = {}
       payload[:lograge][:host] = request.host
       payload[:lograge][:params] = request.filtered_parameters.except(:action, :controller)
       payload[:lograge][:remote_ip] = request.remote_ip
       payload[:lograge][:user_agent] = request.user_agent
       payload[:lograge][:referer] = request.referer
-      payload[:lograge][:controller_action] = "#{params[:controller]}##{params[:action]}" if defined?(params)
       payload[:lograge][:session_id] = session.is_a?(Hash) ? session[:id] : session.id if defined?(session)
       payload[:lograge][:user_id] = current_user&.id if defined?(current_user)
     end
@@ -24,6 +25,15 @@ class GetaroundUtils::Railties::Lograge < Rails::Railtie
     ActionController::Base.prepend LogrageActionController
   end
 
+  HTTP_PARAMS = [:method, :path, :host, :remote_ip,
+                 :status, :duration, :location,
+                 :user_agent, :referer].freeze
+
   config.lograge.enabled = true
-  config.lograge.custom_options = ->(event) { event.payload[:lograge] }
+  config.lograge.custom_options = ->(event) {
+    event.payload[:lograge]
+  }
+  config.lograge.before_format = ->(data, _) {
+    data.except(*HTTP_PARAMS).merge(http: data.slice(*HTTP_PARAMS))
+  }
 end

data/lib/getaround_utils/railties/ougai.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'getaround_utils/ougai/deep_key_value_formatter'
+require 'getaround_utils/ougai/json_formatter'
 require 'request_store'
 require 'rails/railtie'
 require 'ougai'
@@ -38,7 +38,7 @@ class OugaiRequestStoreMiddleware
   end
 
   def call(env)
-    RequestStore.store[:ougai] = { request_id: env['action_dispatch.request_id'] }
+    RequestStore.store[:ougai] = { http: { request_id: env['action_dispatch.request_id'] } }
     @app.call(env)
   end
 end
@@ -46,10 +46,10 @@ end
 class GetaroundUtils::Railties::Ougai < Rails::Railtie
   config.ougai_logger = OugaiRailsLogger.new(STDOUT)
   config.ougai_logger.after_initialize if Rails::VERSION::MAJOR < 6
-  config.ougai_logger.formatter = GetaroundUtils::Ougai::DeepKeyValueFormatter.new
+  config.ougai_logger.formatter = GetaroundUtils::Ougai::JsonFormatter.new
   config.ougai_logger.before_log = lambda do |data|
     request_store = RequestStore.store[:ougai] || {}
-    data.merge!(request_store) if request_store&.any?
+    data.deep_merge!(request_store) if request_store&.any?
 
     sidekiq_context = Thread.current[:sidekiq_context] || {}
     data.merge!(sidekiq: sidekiq_context) if sidekiq_context&.any?

data/lib/getaround_utils/utils.rb

@@ -1,4 +1 @@
-require 'getaround_utils/utils/async_queue'
 require 'getaround_utils/utils/deep_key_value'
-require 'getaround_utils/utils/http_reporter'
-require 'getaround_utils/utils/captur_reporter'

data/lib/getaround_utils/version.rb

@@ -1,3 +1,3 @@
 module GetaroundUtils
-  VERSION = '0.2.11'.freeze
+  VERSION = '0.2.13'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: getaround_utils
 version: !ruby/object:Gem::Version
-  version: 0.2.11
+  version: 0.2.13
 platform: ruby
 authors:
 - Drivy
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-24 00:00:00.000000000 Z
+date: 2020-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -93,14 +93,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.3'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -211,10 +211,7 @@ files:
 - lib/getaround_utils/railties/lograge.rb
 - lib/getaround_utils/railties/ougai.rb
 - lib/getaround_utils/utils.rb
-- lib/getaround_utils/utils/async_queue.rb
-- lib/getaround_utils/utils/captur_reporter.rb
 - lib/getaround_utils/utils/deep_key_value.rb
-- lib/getaround_utils/utils/http_reporter.rb
 - lib/getaround_utils/version.rb
 homepage: https://github.com/drivy
 licenses:

data/lib/getaround_utils/utils/async_queue.rb

@@ -1,49 +0,0 @@
-module GetaroundUtils; end
-module GetaroundUtils::Utils; end
-
-class GetaroundUtils::Utils::AsyncQueue
-  include GetaroundUtils::Mixins::Loggable
-
-  MAX_QUEUE_SIZE = 1000
-  BUFFER_SIZE = 50
-
-  def initialize
-    @queue = []
-    @mutex = Mutex.new
-    @closed = false
-    @worker = Thread.new(&method(:thread_run))
-    at_exit { terminate }
-  end
-
-  def perform
-    raise NotImplementedError
-  end
-
-  def push(payload)
-    @mutex.synchronize do
-      if @queue.size >= MAX_QUEUE_SIZE
-        loggable_log(:error, 'queue overflow')
-      else
-        @queue.push(payload)
-      end
-    end
-  end
-
-  def thread_run
-    loop do
-      buffer = @mutex.synchronize { @queue.shift(BUFFER_SIZE) }
-      loggable_log(:debug, 'thread_run', buffer_size: buffer.size)
-      return if @closed && buffer.empty?
-
-      perform(buffer) unless buffer.empty?
-      sleep(1) unless @mutex.synchronize { @queue.any? }
-    rescue StandardError => e
-      loggable_log(:error, e.message, class: e.class.to_s, backtrace: e.backtrace)
-    end
-  end
-
-  def terminate
-    @mutex.synchronize { @closed = true }
-    @worker&.join
-  end
-end

data/lib/getaround_utils/utils/captur_reporter.rb

@@ -1,32 +0,0 @@
-require 'json'
-require 'faraday'
-require 'getaround_utils/utils/async_queue'
-
-class GetaroundUtils::Utils::CapturReporter < GetaroundUtils::Utils::AsyncQueue
-  CAPTUR_URL = ENV['CAPTUR_URL']
-
-  def perform(events)
-    return unless CAPTUR_URL&.match('^https?://')
-
-    Faraday.post(CAPTUR_URL) do |req|
-      req.options[:open_timeout] = 1
-      req.options[:timeout] = 1
-      req.headers = { 'Content-Type': 'application/json' }
-      req.body = JSON.generate(events: events, metas: metas)
-    end
-  end
-
-  def metas
-    {}
-  end
-
-  def push(uuid:, type:, anonymous_id:, timestamp: nil, attributes: {})
-    super(
-      uuid: uuid,
-      type: type,
-      timestamp: timestamp || Time.now.iso8601,
-      anonymous_id: anonymous_id,
-      attributes: attributes,
-    )
-  end
-end

data/lib/getaround_utils/utils/http_reporter.rb

@@ -1,11 +0,0 @@
-class GetaroundUtils::Utils::HttpReporter
-  include GetaroundUtils::Mixins::Loggable
-
-  def initialize(_)
-    loggable_log(:warn, 'use of deprecated class')
-  end
-
-  def report(_)
-    loggable_log(:warn, 'use of deprecated class')
-  end
-end