time_bandits 0.10.10 → 0.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c58d28c93e2abce5a7e5748cafd1720e71e9efd1a9dbad61dfd84f8c388d18ac
-  data.tar.gz: 4cdd00f82f637c496b5810f46ae1899c898f86850502626c33ab5d676bf8bacd
+  metadata.gz: 5e7b550a5d77896484401846b736c019fba606ae7ab266a14cff054c5f392f61
+  data.tar.gz: d8d58687ebe4687e75ec71b8caf0cba39229a72a50edc614cdd0f0df2200bdbe
 SHA512:
-  metadata.gz: 0b9afbbe602af0a36640c60491f7de0e8bf173619572c65206a2502c2056a05f1b81d9b14c36ebfeacc69e9a7633cc3c53d8c308475dda258d54ffec4f294e57
-  data.tar.gz: fc6dbf5399f916035854330e3a00a517fe50e809cbac3b30d35519c8a74e97eb6b3a8bf0a5c5e9d70f76825d30c69a873f584cacf7952696a427f75384677747
+  metadata.gz: ae0535036dd2df5545890147db397693b8c4afb5c86858d4affaf3f558c5c7a61dd08ece46b47387a280b1283f3f0518dcb916a0d5bd03622ff050b10904175a
+  data.tar.gz: fc61fda4602282fd0a708a695f0ab410f18f9eed931ced47e2aae8177241dda2d72bf2cce903d4cc2fb9ad0abfcebec91da42c1897635bc42c761ad3e1c65fac

data/.travis.yml

@@ -1,11 +1,23 @@
+dist: bionic
 language: ruby
 rvm:
-  - 2.2.5
-  - 2.3.4
-  - 2.4.1
-script: "rake appraisal:test"
-sudo: false
+  - 2.5.7
+  - 2.6.5
+  - 2.7.0
+before_install:
+  - gem install bundler
+script: "bundle exec rake appraisal:test"
 services:
   - redis-server
   - memcached
   - rabbitmq
+  - mysql
+addons:
+  apt:
+    sources:
+      - sourceline: "deb https://packages.erlang-solutions.com/ubuntu bionic contrib"
+        key_url: "https://packages.erlang-solutions.com/ubuntu/erlang_solutions.asc"
+      - sourceline: "deb https://dl.bintray.com/rabbitmq/debian bionic main"
+        key_url: "https://dl.bintray.com/rabbitmq/Keys/rabbitmq-release-signing-key.asc"
+    packages:
+      - rabbitmq-server

data/Appraisals

@@ -1,14 +1,7 @@
 [
-  "4.1.16",
-  "4.2.8",
-  "4.2.9",
-  "5.0.3",
-  "5.0.4",
-  "5.1.1",
-  "5.1.2",
-  "5.1.5",
+  "5.2.4.3",
+  "6.0.3.2"
 ].each do |rails_version|
-  next if RUBY_VERSION >= "2.4.0" && rails_version < "4.2.8"
   appraise "activesupport-#{rails_version}" do
     gem "activesupport", rails_version
     gem "activerecord", rails_version

data/README.md

@@ -53,16 +53,16 @@ With these two new time consumers, the log line changes to
    b: number of bytes allocated by the ruby x_malloc call (#bytes)
    l: live data set size after last GC (#slots)
 
-Sidenote for Germans: you can use the word "Gesabbel" (eng: drivel) as a mnemonic here ;-)
+Side note for speakers of German: you can use the word "Gesabbel" (eng: drivel) as a mnemonic here ;-)
 
-It's realtively straightforward to write additional time consumers; the more difficult part of this is
+It's relatively straightforward to write additional time consumers; the more difficult part of this is
 monkey patching the code which you want to instrument. Have a look at consumers under
-lib/time_bandits/time_consumers and the corresponding patches under lib/time_bandits/monkey_patches.
+`lib/time_bandits/time_consumers` and the corresponding patches under `lib/time_bandits/monkey_patches`.
 
 
 ## Prerequisites
 
-Rails >= 3.x is required. The gem will raise an error if you try to use it with an incompatible
+ActiveSupport/Rails >= 5.2 is required. The gem will raise an error if you try to use it with an incompatible
 version.
 
 You'll need a ruby with the railsexpress GC patches applied, if you want to include GC and heap size
@@ -80,10 +80,37 @@ changed so much of the code that is is practically a full rewrite, hence we chan
 
 ## Running Tests
 
-In order for the test to run you need a running memcached, redis-server and mysql
+Run `docker-compose up` to start Redis, MySQL, RabbitMQ and Memached containers, then run `rake`.
+
 
 ## Release Notes
 
+## Version 0.12.1
+  - support having the rails gem in a bundle without activating the time bandits railtie
+  - automatically install the garbage collection time bandit in rails applications
+
+## Version 0.12.0
+  - drops support for Rails versions before 5.2.0 and Ruby versions before 2.2.0
+  - makes it possible to use individual time bandits without Rails (e.g. in a Sinatra app)
+
+## Version 0.11.0
+  - supports rails 6.0.0
+
+## Version 0.10.12
+  - supports rails 5.0.7
+
+## Version 0.10.11
+  - support rails 5.2.0
+
+## Version 0.10.10
+  - fixed regression introduced in 0.10.9
+
+### Version 0.10.9
+  - added specialized activerecord logging for Rails >= 5.1.5
+
+### Version 0.10.8
+  - rails has changed render_bind in 5.0.3
+
 ### Version 0.10.7
   - fixed bug in activerecord monkeypatch for Rails 5.1
 
@@ -160,7 +187,7 @@ In order for the test to run you need a running memcached, redis-server and mysq
 
 ## License
 
-Copyright (c) 2009-2014 Stefan Kaes <skaes@railsexpress.de>
+Copyright (c) 2009-2020 Stefan Kaes <skaes@railsexpress.de>
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/Rakefile

@@ -13,6 +13,7 @@ Rake::TestTask.new do |t|
   t.libs << "test"
   t.test_files = FileList['test/**/*_test.rb']
   t.verbose = true
+  t.ruby_opts = %w(-W0)
 end
 
 namespace :appraisal do

data/docker-compose.yml

@@ -3,7 +3,7 @@ version: '2'
 services:
   mysql:
     container_name: mysql
-    image: mysql:5.5
+    image: mysql:5.7
     ports:
       - "3601:3306"
     environment:
@@ -11,18 +11,18 @@ services:
 
   memcached:
     container_name: memcached
-    image: memcached:1.4.27
+    image: memcached:1.6.6
     ports:
       - "11211:11211"
 
   redis:
     container_name: redis
-    image: redis:3.2.0
+    image: redis:5.0.9
     ports:
       - "6379:6379"
 
   rabbitmq:
     container_name: rabbitmq
-    image: rabbitmq:3.6.2
+    image: rabbitmq:3.8.5
     ports:
       - "5672:5672"

data/gemfiles/activesupport_5.2.4.3.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "activesupport", "5.2.4.3"
+gem "activerecord", "5.2.4.3"
+
+gemspec path: "../"

data/gemfiles/activesupport_5.2.4.3.gemfile.lock

@@ -0,0 +1,81 @@
+PATH
+  remote: ..
+  specs:
+    time_bandits (0.11.0)
+      activesupport (>= 5.2.4.3)
+      thread_variables
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    activemodel (5.2.4.3)
+      activesupport (= 5.2.4.3)
+    activerecord (5.2.4.3)
+      activemodel (= 5.2.4.3)
+      activesupport (= 5.2.4.3)
+      arel (>= 9.0)
+    activesupport (5.2.4.3)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+    amq-protocol (2.3.1)
+    amqp (1.8.0)
+      amq-protocol (>= 2.2.0)
+      eventmachine
+    ansi (1.5.0)
+    appraisal (2.3.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
+    arel (9.0.0)
+    beetle (3.4.1)
+      activesupport (>= 2.3.4)
+      amq-protocol (= 2.3.1)
+      amqp (= 1.8.0)
+      bunny (~> 0.7.12)
+      hiredis (>= 0.4.5)
+      redis (>= 4.2.1)
+    bunny (0.7.12)
+    byebug (11.1.3)
+    concurrent-ruby (1.1.6)
+    dalli (2.7.10)
+    eventmachine (1.2.7)
+    hiredis (0.6.3)
+    i18n (1.8.3)
+      concurrent-ruby (~> 1.0)
+    memcached (1.8.0)
+    minitest (5.14.1)
+    mocha (1.11.2)
+    mysql2 (0.5.3)
+    rake (13.0.1)
+    redis (4.2.1)
+    sequel (5.33.0)
+    thor (1.0.1)
+    thread_safe (0.3.6)
+    thread_variables (0.2.0)
+    tzinfo (1.2.7)
+      thread_safe (~> 0.1)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  activerecord (= 5.2.4.3)
+  activesupport (= 5.2.4.3)
+  ansi
+  appraisal
+  beetle (>= 3.4.1)
+  byebug
+  dalli
+  memcached (~> 1.8.0)
+  minitest (~> 5.5)
+  mocha
+  mysql2
+  rake
+  redis
+  sequel
+  time_bandits!
+
+BUNDLED WITH
+   2.1.4

data/gemfiles/activesupport_6.0.3.2.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "http://rubygems.org"
+
+gem "activesupport", "6.0.3.2"
+gem "activerecord", "6.0.3.2"
+
+gemspec path: "../"

data/gemfiles/activesupport_6.0.3.2.gemfile.lock

@@ -0,0 +1,81 @@
+PATH
+  remote: ..
+  specs:
+    time_bandits (0.11.0)
+      activesupport (>= 5.2.4.3)
+      thread_variables
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    activemodel (6.0.3.2)
+      activesupport (= 6.0.3.2)
+    activerecord (6.0.3.2)
+      activemodel (= 6.0.3.2)
+      activesupport (= 6.0.3.2)
+    activesupport (6.0.3.2)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+      zeitwerk (~> 2.2, >= 2.2.2)
+    amq-protocol (2.3.1)
+    amqp (1.8.0)
+      amq-protocol (>= 2.2.0)
+      eventmachine
+    ansi (1.5.0)
+    appraisal (2.3.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
+    beetle (3.4.1)
+      activesupport (>= 2.3.4)
+      amq-protocol (= 2.3.1)
+      amqp (= 1.8.0)
+      bunny (~> 0.7.12)
+      hiredis (>= 0.4.5)
+      redis (>= 4.2.1)
+    bunny (0.7.12)
+    byebug (11.1.3)
+    concurrent-ruby (1.1.6)
+    dalli (2.7.10)
+    eventmachine (1.2.7)
+    hiredis (0.6.3)
+    i18n (1.8.3)
+      concurrent-ruby (~> 1.0)
+    memcached (1.8.0)
+    minitest (5.14.1)
+    mocha (1.11.2)
+    mysql2 (0.5.3)
+    rake (13.0.1)
+    redis (4.2.1)
+    sequel (5.33.0)
+    thor (1.0.1)
+    thread_safe (0.3.6)
+    thread_variables (0.2.0)
+    tzinfo (1.2.7)
+      thread_safe (~> 0.1)
+    zeitwerk (2.3.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  activerecord (= 6.0.3.2)
+  activesupport (= 6.0.3.2)
+  ansi
+  appraisal
+  beetle (>= 3.4.1)
+  byebug
+  dalli
+  memcached (~> 1.8.0)
+  minitest (~> 5.5)
+  mocha
+  mysql2
+  rake
+  redis
+  sequel
+  time_bandits!
+
+BUNDLED WITH
+   2.1.4

data/lib/time_bandits.rb

@@ -5,11 +5,7 @@ require 'thread_variables'
 module TimeBandits
 
   module TimeConsumers
-    if defined?(Rails) && Rails::VERSION::STRING < "3.0"
-      autoload :Database,          'time_bandits/time_consumers/database_rails2'
-    else
-      autoload :Database,          'time_bandits/time_consumers/database'
-    end
+    autoload :Database,          'time_bandits/time_consumers/database'
     autoload :GarbageCollection, 'time_bandits/time_consumers/garbage_collection'
     autoload :JMX,               'time_bandits/time_consumers/jmx'
     autoload :MemCache,          'time_bandits/time_consumers/mem_cache'
@@ -20,7 +16,7 @@ module TimeBandits
     autoload :Beetle,            'time_bandits/time_consumers/beetle'
   end
 
-  require 'time_bandits/railtie' if defined?(Rails) && Rails::VERSION::STRING >= "3.0"
+  require 'time_bandits/railtie' if defined?(Rails::Railtie)
   require 'time_bandits/time_consumers/base_consumer'
 
   mattr_accessor :time_bandits

data/lib/time_bandits/monkey_patches/action_controller.rb

@@ -4,45 +4,6 @@ module ActionController #:nodoc:
 
   module Instrumentation
 
-    # patch to ensure that the completed line is always written to the log.
-    # this is not necessary anymore with Rails 4 and higher.
-    def process_action(action, *args)
-      raw_payload = get_raw_payload
-      ActiveSupport::Notifications.instrument("start_processing.action_controller", raw_payload.dup)
-
-      exception = nil
-      result = ActiveSupport::Notifications.instrument("process_action.action_controller", raw_payload) do |payload|
-        begin
-          super
-        rescue Exception => exception
-          response.status = 500
-          nil
-        ensure
-          payload[:status] = response.status
-          append_info_to_payload(payload)
-        end
-      end
-      raise exception if exception
-      result
-    end unless Rails::VERSION::STRING >= "4.0"
-
-    # patch to ensure that render times are always recorded in the log.
-    # this is not necessary anymore with Rails 3 and up.
-    def render(*args)
-      render_output = nil
-      exception = nil
-      self.view_runtime = cleanup_view_runtime do
-        Benchmark.ms do
-          begin
-            render_output = super
-          rescue Exception => exception
-          end
-        end
-      end
-      raise exception if exception
-      render_output
-    end unless Rails::VERSION::STRING >= "3.0"
-
     def cleanup_view_runtime #:nodoc:
       consumed_before_rendering = TimeBandits.consumed
       runtime = yield
@@ -52,32 +13,6 @@ module ActionController #:nodoc:
 
     private
 
-    if Rails::VERSION::STRING =~ /\A3\.[01]/
-      def get_raw_payload
-        {
-          :controller => self.class.name,
-          :action     => self.action_name,
-          :params     => request.filtered_parameters,
-          :formats    => request.formats.map(&:to_sym),
-          :method     => request.method,
-          :path       => (request.fullpath rescue "unknown")
-        }
-      end
-    elsif Rails::VERSION::STRING =~ /\A3\.2/
-      def get_raw_payload
-        {
-          :controller => self.class.name,
-          :action     => self.action_name,
-          :params     => request.filtered_parameters,
-          :format     => request.format.try(:ref),
-          :method     => request.method,
-          :path       => (request.fullpath rescue "unknown")
-        }
-      end
-    elsif Rails::VERSION::STRING < "3"
-      raise "time_bandits ActionController monkey patch is not compatible with your Rails version"
-    end
-
     module ClassMethods
       # patch to log rendering time with more precision
       def log_process_action(payload) #:nodoc:
@@ -104,7 +39,7 @@ module ActionController #:nodoc:
 
       # this is an ugly hack to ensure completed lines show up in the test logs
       # TODO: move this code to some other place
-      return unless Rails.env.test? && Rails::VERSION::STRING >= "3.2"
+      return unless Rails.env.test?
 
       status = payload[:status]
       if status.nil? && payload[:exception].present?
@@ -118,7 +53,7 @@ module ActionController #:nodoc:
     end
   end
 
-  # this gets included in ActionController::Base
+  # this gets included in ActionController::Base in the time_bandits railtie
   module TimeBanditry #:nodoc:
     extend ActiveSupport::Concern