informant-rails 2.0.1 → 2.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 508ce7cb53f750ba6b422a41c966b1fbdd46b623dcb311c4410d80c9b3b95f2f
-  data.tar.gz: 28662decb01dd5ae5b943a0f099bf68eaf3bbbfd1b6cb31172eb7d1673238b24
+  metadata.gz: 3ea25418001e3eab7a6a002c400905fae2c59e43cc43c31897bf04aac59ec499
+  data.tar.gz: a2f87283669067d6c79e4c9f892f47e20a7398a42f216fdfd0cecc93ec3812ae
 SHA512:
-  metadata.gz: babd9a69b4121f59f3dae1bddc3933d33c083d2f86ad8cc95cd265b8dc7c13c795b8f6803c81764c91a0868357d27f11863bac12c9e83b55bbfbfbfb8f6f2010
-  data.tar.gz: ec774a339d41ba6c95f82bb37f78eb90280f4c238dcbf4b31f1a7b71abdc5c3ee05bbddeb888e99232cffdceb4a112df59de270c7963623deb130b4eb1765ed0
+  metadata.gz: '07936249ff1fecbadddf07edf8f3c2da63b421fd4a29040f27644809ccfe9e7818cf247e675f7ed36635ba5f45efcf017649389e4deb28090f0e2735764b01bd'
+  data.tar.gz: f99902c5bed9ad84be24befce881a1d0c29ec632f67166a3a6ed39141e8305883d43c2f7ec34fe3b37397f758d2262e0f577d3dd06e790020ebd26a5b6a78338

data/README.markdown

@@ -8,12 +8,11 @@ The informant-rails gem provides Rails and ActiveRecord hooks for The Informant.
 
 ## Compatibility
 
-The informant-rails gem officially supports Ruby 2.0+.
+The informant-rails gem officially supports Ruby 2.5+.
 
-It will work automatically with Rails 3 and up as well as Mongoid 3 and up.
+It will work automatically with Rails 5.2+ as well as Mongoid 6.2 and up.
 
-[![Build Status](https://travis-ci.org/informantapp/informant-rails.svg?branch=master)](https://travis-ci.org/informantapp/informant-rails)
-[![Code Climate](https://codeclimate.com/github/informantapp/informant-rails.png)](https://codeclimate.com/github/informantapp/informant-rails)
+[![pipeline status](https://gitlab.com/informantapp/informant-rails/badges/master/pipeline.svg)](https://gitlab.com/informantapp/informant-rails/-/commits/master)
 
 ## Installation
 
@@ -28,6 +27,10 @@ It will work automatically with Rails 3 and up as well as Mongoid 3 and up.
 4. Deploy with the gem installed
 5. Submit a form and you'll see it appear in our web interface
 
+We've set up
+[informant-rails-example](https://gitlab.com/informantapp/informant-rails-example)
+as a demonstrate of installation and operation.
+
 ## Usage
 
 By default, any request that causes an ActiveRecord or Mongoid model to be validated will be tracked by the Informant once the gem is added to your project.

data/Rakefile

@@ -1,12 +1,15 @@
-require "bundler/setup"
-require "bundler/gem_tasks"
+require 'bundler/setup'
+require 'bundler/gem_tasks'
 
-if !ENV["APPRAISAL_INITIALIZED"] && !ENV["TRAVIS"]
-  require "appraisal/task"
-  Appraisal::Task.new
-  task default: :appraisal
-else
-  require "rspec/core/rake_task"
+default_task = :spec
+
+if ENV['APPRAISAL_INITIALIZED']
+  require 'rspec/core/rake_task'
   RSpec::Core::RakeTask.new
-  task default: :spec
+else
+  require 'appraisal/task'
+  Appraisal::Task.new
+  default_task = :appraisal
 end
+
+task default: default_task

data/lib/informant-rails/config.rb

@@ -1,24 +1,13 @@
-module InformantRails::Config
-  extend self
-
-  attr_accessor :api_token, :exclude_models, :filter_parameters, :value_tracking
-
-  self.api_token
-  self.exclude_models = []
-  self.filter_parameters = []
-  self.value_tracking = true
-
-  def configure; yield self end
-
-  def self.client_identifier
-    @client_identifier ||= "informant-rails-#{InformantRails::VERSION}"
-  end
-
-  def self.enabled?
-    api_token.present?
-  end
-
-  def self.collector_host
-    @collector_host ||= ENV['INFORMANT_COLLECTOR_HOST'] || 'https://collector-api.informantapp.com'
+module InformantRails
+  module Config
+    extend self
+
+    delegate :api_token, :api_token=,
+             :collector_host,
+             :enabled?,
+             :exclude_models, :exclude_models=,
+             :filter_parameters, :filter_parameters=,
+             :value_tracking?, :value_tracking=,
+             to: InformantCommon::Config
   end
 end

data/lib/informant-rails/diagnostic.rb

@@ -1,24 +1,23 @@
+class TestClass
+  include ActiveModel::Model
+
+  attr_accessor :field_name
+
+  validates_presence_of :field_name
+end
+
 module InformantRails
   class Diagnostic
-    def self.run; new.run end
+    def self.run
+      new.run
+    end
 
     def run
       if Config.api_token.blank?
         Rails.logger.info missing_api_token_message
       else
-        Client.record('REQUEST_METHOD' => 'POST')
-        Client.record_action('Connectivity', 'test')
-        Client.request.instance_variable_set('@models', [{
-          name: 'TestClass',
-          errors: [name: 'field_name', value: 'field_value', message: 'must be unique']
-        }])
-        response = Client.transmit(Client.request)
-
-        if response.code == '204'
-          Rails.logger.info success_message
-        else
-          Rails.logger.info bad_response_message(response.body)
-        end
+        response = InformantCommon::Client.transmit(test_form_submission).join.value
+        display_response_message(response)
       end
 
       Rails.logger.info assistance_message
@@ -44,11 +43,26 @@ module InformantRails
     end
 
     def success_message
-      "Everything looks good. You should see a test request on your dashboard."
+      'Everything looks good. You should see a test request on your dashboard.'
     end
 
     def assistance_message
       "If you need assistance or have any questions, send an email to support@informantapp.com or tweet @informantapp and we'll help you out!"
     end
+
+    def test_form_submission
+      InformantCommon::Event::FormSubmission.new.tap do |form_submission|
+        form_submission.handler = 'Connectivity#test'
+        form_submission.process_model(InformantCommon::Model::ActiveModel.new(TestClass.new.tap(&:valid?)))
+      end
+    end
+
+    def display_response_message(response)
+      if response.code == '204'
+        Rails.logger.info success_message
+      else
+        Rails.logger.info bad_response_message(response.body)
+      end
+    end
   end
 end

data/lib/informant-rails/middleware.rb

@@ -1,9 +1,13 @@
 module InformantRails
-  class Middleware < Struct.new(:app)
+  class Middleware
+    def initialize(app)
+      @app = app
+    end
+
     def call(env)
-      Client.record(env)
-      response = app.call(env)
-      Client.process
+      InformantCommon::Client.start_transaction!(env['REQUEST_METHOD'])
+      response = @app.call(env)
+      InformantCommon::Client.process
       response
     end
   end

data/lib/informant-rails/railtie.rb

@@ -1,30 +1,31 @@
 module InformantRails
   class Railtie < ::Rails::Railtie
     initializer 'informant' do |config|
-      Config.api_token ||= ENV['INFORMANT_API_KEY']
+      if Config.enabled?
+        InformantCommon::Client.transmit(
+          InformantCommon::Event::AgentInfo.new(
+            agent_identifier: "informant-rails-#{InformantRails::VERSION}",
+            framework_version: "rails-#{Rails.version}"
+          )
+        )
+        InformantRails::Config.filter_parameters = Rails.configuration.filter_parameters
 
-      begin
-        if Config.enabled?
-          InformantRails::Client.log_client_info
-          InformantRails::Config.filter_parameters = Rails.configuration.filter_parameters
+        config.middleware.use InformantRails::Middleware
 
-          config.middleware.use InformantRails::Middleware
-
-          ActiveSupport.on_load(:action_controller) do
-            include InformantRails::RequestTracking
-          end
+        ActiveSupport.on_load(:action_controller) do
+          include InformantRails::RequestTracking
+        end
 
-          ActiveSupport.on_load(:active_record) do
-            include InformantRails::ValidationTracking
-          end
+        ActiveSupport.on_load(:active_record) do
+          include InformantRails::ValidationTracking
+        end
 
-          ActiveSupport.on_load(:mongoid) do
-            include InformantRails::ValidationTracking
-          end
+        ActiveSupport.on_load(:mongoid) do
+          include InformantRails::ValidationTracking
         end
-      rescue
-        puts "Unable to bootstrap informant."
       end
+    rescue StandardError => e
+      puts "Unable to bootstrap informant: #{e.message}"
     end
   end
 end

data/lib/informant-rails/request_tracking.rb

@@ -5,11 +5,11 @@ module InformantRails
     included do
       if defined? before_action
         before_action do
-          InformantRails::Client.record_action(controller_name, action_name)
+          InformantCommon::Client.current_transaction&.handler = [controller_name, action_name].join('#')
         end
       else
         before_filter do
-          InformantRails::Client.record_action(controller_name, action_name)
+          InformantCommon::Client.current_transaction&.handler = [controller_name, action_name].join('#')
         end
       end
     end

data/lib/informant-rails/validation_tracking.rb

@@ -3,9 +3,7 @@ module InformantRails
     extend ActiveSupport::Concern
 
     included do
-      set_callback(:validate, :after) do
-        InformantRails::Client.record_validated_model(self)
-      end
+      include InformantCommon::ValidationTracking
     end
   end
 end

data/lib/informant-rails/version.rb

@@ -1,3 +1,3 @@
 module InformantRails
-  VERSION = '2.0.1'
+  VERSION = '2.4.0'.freeze
 end

data/lib/informant-rails.rb

@@ -3,23 +3,15 @@ if defined?(Rake)
   Dir[File.join(File.dirname(__FILE__), 'tasks', '**/*.rake')].each { |rake| load rake }
 end
 
+require 'informant-common'
+
 module InformantRails
   autoload :Config,             'informant-rails/config'
   autoload :Diagnostic,         'informant-rails/diagnostic'
-  autoload :Client,             'informant-rails/client'
-  autoload :FieldError,         'informant-rails/field_error'
   autoload :Middleware,         'informant-rails/middleware'
-  autoload :Model,              'informant-rails/model'
-  autoload :ParameterFilter,    'informant-rails/parameter_filter'
   autoload :RequestTracking,    'informant-rails/request_tracking'
   autoload :ValidationTracking, 'informant-rails/validation_tracking'
   autoload :VERSION,            'informant-rails/version'
-
-  module Event
-    autoload :Base,             'informant-rails/event/base'
-    autoload :ClientInfo,       'informant-rails/event/client_info'
-    autoload :FormSubmission,   'informant-rails/event/form_submission'
-  end
 end
 
 require 'informant-rails/railtie'

data/lib/tasks/diagnostic.rake

@@ -1,9 +1,7 @@
 namespace :informant do
-
-  desc "Verify connectivity from your app to Informant"
-  task :diagnostic => :environment do
-    Rails.logger = Logger.new(STDOUT)
+  desc 'Verify connectivity from your app to Informant'
+  task diagnostic: :environment do
+    Rails.logger = Logger.new($stdout)
     InformantRails::Diagnostic.run
   end
-
 end

metadata

@@ -1,35 +1,63 @@
 --- !ruby/object:Gem::Specification
 name: informant-rails
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.4.0
 platform: ruby
 authors:
 - Informant, LLC
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-03-01 00:00:00.000000000 Z
+date: 2021-12-18 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: informant-common
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 5.2.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0
+        version: 8.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 5.2.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0
+        version: 8.0.0
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  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'
 description: The Informant tracks what users do wrong in your forms so you can make
   them better.
 email:
@@ -42,16 +70,9 @@ files:
 - README.markdown
 - Rakefile
 - lib/informant-rails.rb
-- lib/informant-rails/client.rb
 - lib/informant-rails/config.rb
 - lib/informant-rails/diagnostic.rb
-- lib/informant-rails/event/base.rb
-- lib/informant-rails/event/client_info.rb
-- lib/informant-rails/event/form_submission.rb
-- lib/informant-rails/field_error.rb
 - lib/informant-rails/middleware.rb
-- lib/informant-rails/model.rb
-- lib/informant-rails/parameter_filter.rb
 - lib/informant-rails/railtie.rb
 - lib/informant-rails/request_tracking.rb
 - lib/informant-rails/validation_tracking.rb
@@ -60,8 +81,9 @@ files:
 homepage: https://www.informantapp.com
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -69,16 +91,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.8
-signing_key: 
+rubygems_version: 3.2.32
+signing_key:
 specification_version: 4
 summary: The Informant tracks server-side validation errors and gives you metrics
   you never dreamed of.

data/lib/informant-rails/client.rb

@@ -1,63 +0,0 @@
-require 'net/http'
-
-module InformantRails
-  class Client
-
-    def self.supported_request_methods
-      @supported_request_methods ||= %w(POST PATCH PUT DELETE)
-    end
-
-    def self.record(env)
-      new_request if supported_request_methods.include?(env['REQUEST_METHOD'])
-    end
-
-    def self.record_action(controller_name, action)
-      if request
-        request.filename = controller_name
-        request.action = action
-      end
-    end
-
-    def self.record_validated_model(model)
-      request.process_model(model) if request && include_model?(model)
-    end
-
-    def self.process
-      if request && request.models.any?
-        this_request = request
-        Thread.new { transmit(this_request) }
-      end
-    ensure
-      cleanup
-    end
-
-    def self.request
-      Thread.current[:informant_request]
-    end
-
-    def self.cleanup
-      Thread.current[:informant_request] = nil
-    end
-
-    def self.transmit(event)
-      Net::HTTP.start(*event.net_http_start_arguments) do |http|
-        http.request(event.post_request)
-      end
-    end
-
-    def self.log_client_info
-      transmit(InformantRails::Event::ClientInfo.new)
-    end
-
-    private
-
-    def self.include_model?(model)
-      !Config.exclude_models.include?(model.class.to_s)
-    end
-
-    def self.new_request
-      Thread.current[:informant_request] = Event::FormSubmission.new
-    end
-
-  end
-end

data/lib/informant-rails/event/base.rb

@@ -1,25 +0,0 @@
-module InformantRails::Event
-  class Base
-    def self.endpoint
-      raise 'Must implement'
-    end
-    def endpoint; self.class.endpoint end
-
-    def self.authorization_header_value
-      @authorization_header_value ||= ActionController::HttpAuthentication::Token.encode_credentials(InformantRails::Config.api_token)
-    end
-    def authorization_header_value; self.class.authorization_header_value end
-
-    def post_request
-      Net::HTTP::Post.new(endpoint, {
-        "Authorization" => authorization_header_value,
-        "Content-Type" => "application/json"
-      }).tap { |r| r.body = to_json }
-    end
-
-    def self.net_http_start_arguments
-      @net_http_start_arguments ||= [endpoint.host, endpoint.port, use_ssl: endpoint.scheme == 'https']
-    end
-    def net_http_start_arguments; self.class.net_http_start_arguments end
-  end
-end

data/lib/informant-rails/event/client_info.rb

@@ -1,16 +0,0 @@
-module InformantRails::Event
-  class ClientInfo < Base
-    def as_json(*args)
-      {
-        agent_identifier: InformantRails::Config.client_identifier,
-        framework_version: "rails-#{Rails.version}",
-        runtime_version: "ruby-#{RUBY_VERSION}",
-        mongoid_version: defined?(Mongoid) ? Mongoid::VERSION : nil
-      }
-    end
-
-    def self.endpoint
-      @endpoint ||= URI("#{InformantRails::Config.collector_host}/v2/client-info")
-    end
-  end
-end

data/lib/informant-rails/event/form_submission.rb

@@ -1,36 +0,0 @@
-module InformantRails::Event
-  class FormSubmission < Base
-    attr_accessor :request_url, :filename, :action
-
-    def process_model(model)
-      if model && untracked?(model)
-        models << InformantRails::Model.new(model)
-      end
-    end
-
-    def models
-      @models ||= []
-    end
-
-    def as_json(*args)
-      {
-        name: [filename, action].compact.join('#'),
-        models: models.map(&:as_json)
-      }
-    end
-
-    def self.endpoint
-      @endpoint ||= URI("#{InformantRails::Config.collector_host}/v2/form-submissions")
-    end
-
-    protected
-
-    def rails_version
-      @rails_version ||= Rails.version
-    end
-
-    def untracked?(model)
-      !models.detect { |container| container.model == model }
-    end
-  end
-end

data/lib/informant-rails/field_error.rb

@@ -1,19 +0,0 @@
-module InformantRails
-  class FieldError
-    attr_accessor :name, :value, :message
-
-    def initialize(name, value, message=nil)
-      self.name = name
-      self.value = value
-      self.message = message
-    end
-
-    def value=(value)
-      @value = InformantRails::ParameterFilter.filter(name, value)
-    end
-
-    def as_json(*args)
-      { name: name, value: value, message: message }
-    end
-  end
-end

data/lib/informant-rails/model.rb

@@ -1,17 +0,0 @@
-module InformantRails
-  class Model < Struct.new(:model)
-    def name
-      model.class.name
-    end
-
-    def errors
-      model.errors.map do |field, error|
-        InformantRails::FieldError.new(field.to_s, model[field], error)
-      end
-    end
-
-    def as_json(*args)
-      { name: name, errors: errors.map(&:as_json) }
-    end
-  end
-end

data/lib/informant-rails/parameter_filter.rb

@@ -1,13 +0,0 @@
-module InformantRails
-  class ParameterFilter
-    def self.filter(name, value)
-      Config.value_tracking && !matcher.match(name) ? value : '[FILTERED]'
-    end
-
-    protected
-
-    def self.matcher
-      @matcher ||= Regexp.new(/#{Config.filter_parameters.join('|').presence || '$^'}/)
-    end
-  end
-end