activerecord-instrumentation 0.4.0.jlauer3 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8198b8d854e91852b7ade88c96b024922a5e26bbb5bcaf7101f15544ab965762
-  data.tar.gz: 39eb3c1ec55a89e3b49729d64f48c23f4aa8edc4cc41e634f9d06221c8b7e8e9
+  metadata.gz: ec1aa8f04a43ddc425d1b75bbb1b36e2a460a40301fa26ab3c231011acb2205a
+  data.tar.gz: 584f05b61f5bc532dcdbf65cd08d5e45697994f8dec83ab9c2f85b02175c0b9e
 SHA512:
-  metadata.gz: b15c68aae2cc181dd503cd27252dfba14c053cb90f09c1599b5c5a1b5da2ef8d931668a96207f6de328f5f6637c20ae02b8a8f5e49d9d43c8e627d7ca17a7b5a
-  data.tar.gz: 4e87fadf9dcaeaabb238eeb2453d63d80df1e4030763f46ffd7df4a8d30f5ad942c5297c8a98a45a63cb73310f5e0a54ffda5c8e414351c05141248865be8563
+  metadata.gz: 9f4f2a6b39886d381866f1df1e0cd0edd83c39c9f603139948276216e8627cc34a9db7be1a10a8dfdb2df660601f28e291de642a9207ffbfefc78176ca96c570
+  data.tar.gz: 38c5ba6648ade1f097333ce41e4b47b6ecca143ad620b96b60cae1d6c68886d10726f96039040826b71b09463fb95ffa620aead8389375777af84d4c71503f78

data/.circleci/config.yml

@@ -7,7 +7,7 @@ executors:
   ruby-latest:
     resource_class: small
     docker:
-      - image: circleci/ruby:latest
+      - image: circleci/ruby:2.7
         environment:
           BUNDLE_VERSION: "~> 1.17"
 
@@ -45,7 +45,7 @@ jobs:
             - v1-bundle-{{ checksum "Gemfile.lock" }}-
       - run:
           name: Install Ruby Dependencies
-          command: bundle check --path=vendor/bundle || bundle install --local --frozen --path=vendor/bundle --jobs=4 --retry=3 && bundle clean
+          command: bundle install --path=vendor/bundle # install every time so we don't have to package vendor/bundle with the gem
       - save_cache:
           key: v1-bundle-{{ checksum "Gemfile.lock" }}-
           paths:
@@ -123,4 +123,3 @@ workflows:
           requires:
             - gem-build
           context: artifact_publishing
-          

data/CHANGELOG.md

@@ -1,10 +1,17 @@
 Changelog
 =========
 
-## 0.4.0.jlauer3 04/27/2020
+## 0.5.1 01/20/2022
+  * Fixed the publishing process
+
+## 0.5.0 01/18/2022
+  * Added db.query_type and db.query_category tags
+  * Added forwards compatibility code for the connection_config for rails 6.2
+
+## 0.4.0 04/28/2020
   * Add SQL sanitizers
 
 ## 0.3.0 04/22/2020
   * Set up build pipeline with circleci and gem-publisher
   * Fixed linting issues
-  * Renamed gem to `activerecord-instrumentation`
+  * Renamed gem to `activerecord-instrumentation`

data/Gemfile

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-# BEFORE changing this file, read https://wiki.doximity.com/articles/gemfile-maintenance
-source "https://artifacts.dox.support/repository/gems"
+source "https://rubygems.org"
 
 gemspec

data/Gemfile.lock

@@ -1,43 +1,37 @@
 PATH
   remote: .
   specs:
-    activerecord-instrumentation (0.4.0.jlauer3)
+    activerecord-instrumentation (0.5.1)
       activerecord
       opentracing (~> 0.5)
 
 GEM
-  remote: https://artifacts.dox.support/repository/gems/
+  remote: https://rubygems.org/
   specs:
-    activemodel (6.0.2.2)
-      activesupport (= 6.0.2.2)
-    activerecord (6.0.2.2)
-      activemodel (= 6.0.2.2)
-      activesupport (= 6.0.2.2)
-    activesupport (6.0.2.2)
+    activemodel (6.1.4.4)
+      activesupport (= 6.1.4.4)
+    activerecord (6.1.4.4)
+      activemodel (= 6.1.4.4)
+      activesupport (= 6.1.4.4)
+    activesupport (6.1.4.4)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
     ast (2.4.0)
-    concurrent-ruby (1.1.6)
+    concurrent-ruby (1.1.9)
     diff-lcs (1.3)
-    dox-style (1.3.0)
-      rubocop (~> 0.77)
-      rubocop-performance (~> 1.5.1)
-      rubocop-rails (~> 2.2.0)
-      rubocop-rspec (~> 1.37.0)
-    i18n (1.8.2)
+    i18n (1.8.11)
       concurrent-ruby (~> 1.0)
     jaro_winkler (1.5.4)
-    minitest (5.14.0)
+    minitest (5.15.0)
     opentracing (0.5.0)
     opentracing_test_tracer (0.1.1)
       opentracing
     parallel (1.19.1)
     parser (2.7.1.1)
       ast (~> 2.4.0)
-    rack (2.2.2)
     rainbow (3.0.0)
     rake (13.0.1)
     rdoc (6.2.1)
@@ -63,22 +57,16 @@ GEM
       rainbow (>= 2.2.2, < 4.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 1.7)
-    rubocop-performance (1.5.2)
-      rubocop (>= 0.71.0)
-    rubocop-rails (2.2.1)
-      rack (>= 1.1)
-      rubocop (>= 0.72.0)
     rubocop-rspec (1.37.1)
       rubocop (>= 0.68.1)
     ruby-progressbar (1.10.1)
     sdoc (1.1.0)
       rdoc (>= 5.0)
     sqlite3 (1.4.2)
-    thread_safe (0.3.6)
-    tzinfo (1.2.7)
-      thread_safe (~> 0.1)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
     unicode-display_width (1.6.1)
-    zeitwerk (2.3.0)
+    zeitwerk (2.5.3)
 
 PLATFORMS
   ruby
@@ -86,7 +74,6 @@ PLATFORMS
 DEPENDENCIES
   activerecord-instrumentation!
   bundler (~> 1.17)
-  dox-style
   opentracing_test_tracer (~> 0.1)
   rake (~> 13.0)
   rspec (~> 3.9.0)

data/activerecord-instrumentation.gemspec

@@ -23,7 +23,6 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_development_dependency "bundler", "~> 1.17"
-  spec.add_development_dependency "dox-style"
   spec.add_development_dependency "opentracing_test_tracer", "~> 0.1"
   spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "rspec", "~> 3.9.0"

data/lib/active_record/open_tracing/processor.rb

@@ -8,6 +8,35 @@ module ActiveRecord
       SPAN_KIND = "client"
       DB_TYPE = "sql"
 
+      # Used to guess what type of query is running based on the first word of the query
+      #
+      # Categories are
+      # table: Run an action against a table changes the table metadata or configuration
+      # read: Read from the database
+      # write: Write or delete records to the database
+      # unknown: Can't tell the query action from the first word of the query
+      # not_found: First word of the query is not in this list
+      QUERY_CATEGORIES = {
+        alter: "table",
+        call: "unknown", # run a subquery
+        create: "table",
+        delete: "write",
+        drop: "table",
+        do: "read",
+        handler: "table", # table metadata
+        import: "write",
+        insert: "write",
+        load: "write", # covers LOAD XML and LOAD DATA queries
+        rename: "table",
+        replace: "write", # insert, on duplicate overwrite
+        select: "read",
+        table: "read", # similar to select
+        truncate: "table",
+        update: "write",
+        values: "unknown", # generates rows to use as a table but doesn't hit the database
+        with: "unknown" # sets up subqueries in preparation for other queries
+      }
+
       attr_reader :tracer, :sanitizer, :sql_logging_enabled
 
       def initialize(tracer, sanitizer: nil, sql_logging_enabled: true)
@@ -54,9 +83,22 @@ module ActiveRecord
         }.merge(db_statement(payload))
       end
 
+      # rubocop:disable Metrics/MethodLength
       def db_statement(payload)
-        sql_logging_enabled ? { "db.statement" => sanitize_sql(payload.fetch(:sql).squish) } : {}
+        if sql_logging_enabled
+          query_sql = sanitize_sql(payload.fetch(:sql).squish)
+          first_word = query_sql.split.first.downcase
+
+          {
+            "db.statement" => query_sql,
+            "db.query_type" => first_word,
+            "db.query_category" => QUERY_CATEGORIES[first_word.to_sym] || "not_found"
+          }
+        else
+          {}
+        end
       end
+      # rubocop:enable Metrics/MethodLength
 
       def sanitize_sql(sql)
         sanitizer ? sanitizer.sanitize(sql) : sql
@@ -76,7 +118,9 @@ module ActiveRecord
       end
 
       def connection_config
-        @connection_config ||= ActiveRecord::Base.connection_config
+        # Rails 6.2 will deprecate ActiveRecord::Base.connection_config
+        @connection_config ||=
+          ActiveRecord::Base.try(:connection_db_config)&.configuration_hash || ActiveRecord::Base.connection_config
       end
     end
   end

data/lib/active_record/open_tracing/sql_sanitizer.rb

@@ -19,7 +19,7 @@ module ActiveRecord
 
       class << self
         def build_sanitizer(sanitizer_name)
-          sanitizer_klass(sanitizer_name).build
+          sanitizer_klass(sanitizer_name).new
         end
 
         private

data/lib/active_record/open_tracing/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRecord
   module OpenTracing
-    VERSION = "0.4.0.jlauer3"
+    VERSION = "0.5.1"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-instrumentation
 version: !ruby/object:Gem::Version
-  version: 0.4.0.jlauer3
+  version: 0.5.1
 platform: ruby
 authors:
 - SaleMove TechMovers
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-28 00:00:00.000000000 Z
+date: 2022-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -25,20 +25,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.17'
-- !ruby/object:Gem::Dependency
-  name: dox-style
-  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: opentracing_test_tracer
   requirement: !ruby/object:Gem::Requirement
@@ -223,11 +209,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.32
 signing_key: 
 specification_version: 4
 summary: ActiveRecord OpenTracing intrumenter