influxdb-rails 1.0.0.beta3 → 1.0.1.beta3

data/.travis.yml

@@ -1,37 +0,0 @@
-sudo: required
-dist: trusty
-language: ruby
-before_install:
-  - gem update --system --no-doc
-  - gem install bundler --no-doc
-  - gem update bundler --no-doc
-rvm:
-  - ruby-head
-  - "2.6"
-  - "2.5"
-  - "2.4"
-  - "2.3"
-gemfile:
-  - gemfiles/Gemfile.rails-5.2.x
-  - gemfiles/Gemfile.rails-5.1.x
-  - gemfiles/Gemfile.rails-5.0.x
-  - gemfiles/Gemfile.rails-4.2.x
-env:
-  - TEST_TASK=spec
-matrix:
-  allow_failures:
-    - rvm: ruby-head
-  include:
-    - { rvm: "2.6",   gemfile: "Gemfile", env: [TEST_TASK=rubocop] }
-  exclude:
-    # Rails < 5 not on MRI 2.4+
-    - { rvm: ruby-head, gemfile: "gemfiles/Gemfile.rails-4.2.x" }
-    - { rvm: "2.6",   gemfile: "gemfiles/Gemfile.rails-4.2.x" }
-    - { rvm: "2.5",   gemfile: "gemfiles/Gemfile.rails-4.2.x" }
-    - { rvm: "2.4",   gemfile: "gemfiles/Gemfile.rails-4.2.x" }
-addons:
-  apt:
-    packages:
-      - haveged
-      - libgmp-dev
-script: bundle exec rake $TEST_TASK

data/gemfiles/Gemfile.rails-4.2.x

@@ -1,7 +0,0 @@
-source "https://rubygems.org"
-
-gem 'actionpack', '~> 4.2.0'
-gem 'activesupport', '~> 4.2.0'
-gem 'activemodel', '~> 4.2.0'
-
-gemspec :path => '../'

data/gemfiles/Gemfile.rails-5.0.x

@@ -1,7 +0,0 @@
-source "https://rubygems.org"
-
-gem 'actionpack', '~> 5.0.0'
-gem 'activesupport', '~> 5.0.0'
-gem 'activemodel', '~> 5.0.0'
-
-gemspec :path => '../'

data/gemfiles/Gemfile.rails-5.1.x

@@ -1,7 +0,0 @@
-source "https://rubygems.org"
-
-gem 'actionpack', '~> 5.1.0'
-gem 'activesupport', '~> 5.1.0'
-gem 'activemodel', '~> 5.1.0'
-
-gemspec :path => '../'

data/lib/influxdb/rails/air_traffic_controller.rb

@@ -1,41 +0,0 @@
-module InfluxDB
-  module Rails
-    module AirTrafficController # :nodoc:
-      def influxdb_request_data # rubocop:disable Metrics/MethodLength
-        {
-          params:       influxdb_filtered_params,
-          session_data: influxdb_session_data,
-          controller:   params[:controller],
-          action:       params[:action],
-          request_url:  influxdb_request_url,
-          user_agent:   request.env["HTTP_USER_AGENT"],
-          remote_ip:    request.remote_ip,
-          referer:      request.referer,
-          current_user: (current_user rescue nil),
-        }
-      end
-
-      private
-
-      def influxdb_session_data
-        session.respond_to?(:to_hash) ? session.to_hash : session.data
-      end
-
-      def influxdb_request_url
-        url = "#{request.protocol}#{request.host}"
-        url << ":#{request.port}" unless [80, 443].include?(request.port)
-        url << request.fullpath
-      end
-
-      def influxdb_filtered_params
-        if respond_to?(:filter_parameters)
-          filter_parameters(unfiltered_params)
-        elsif defined?(request.filtered_parameters)
-          request.filtered_parameters
-        else
-          params.to_hash.except(:password, :password_confirmation)
-        end
-      end
-    end
-  end
-end

data/lib/influxdb/rails/backtrace.rb

@@ -1,44 +0,0 @@
-module InfluxDB
-  module Rails
-    class Backtrace # rubocop:disable Style/Documentation
-      class Line # rubocop:disable Style/Documentation
-        FORMAT = /^((?:[a-zA-Z]:)?[^:]+):(\d+)(?::in `([^']+)')?$/.freeze
-
-        attr_reader :file
-        attr_reader :number
-        attr_reader :method
-
-        def initialize(line)
-          _, @file, @number, @method = line.match(FORMAT).to_a
-        end
-
-        def to_s
-          "#{file}:#{number} in `#{method}'"
-        end
-
-        def inspect
-          "<Line: #{to_s}>" # rubocop:disable Lint/StringConversionInInterpolation
-        end
-      end
-
-      attr_reader :lines
-
-      def initialize(backtrace)
-        @lines = Array(backtrace).each.collect do |line|
-          InfluxDB::Rails.configuration.backtrace_filters.each do |filter|
-            line = filter.call(line)
-          end
-          Line.new(line)
-        end
-      end
-
-      def to_a
-        lines.map(&:to_s)
-      end
-
-      def inspect
-        "<Backtrace: " + lines.collect(&:to_s).join(", ") + ">"
-      end
-    end
-  end
-end

data/lib/influxdb/rails/exception_presenter.rb

@@ -1,94 +0,0 @@
-require "base64"
-require "socket"
-require "json"
-
-module InfluxDB
-  module Rails
-    class ExceptionPresenter # rubocop:disable Style/Documentation
-      attr_reader :exception
-      attr_reader :backtrace
-      attr_reader :params
-      attr_reader :session_data
-      attr_reader :current_user
-      attr_reader :controller
-      attr_reader :action
-      attr_reader :request_url
-      attr_reader :referer
-      attr_reader :remote_ip
-      attr_reader :user_agent
-      attr_reader :custom_data
-
-      def initialize(ex, params = {})
-        ex = ex.continued_exception if ex.respond_to?(:continued_exception)
-        ex = ex.original_exception if ex.respond_to?(:original_exception)
-
-        @exception    = ex.is_a?(String) ? Exception.new(ex) : ex
-        @backtrace    = InfluxDB::Rails::Backtrace.new(@exception.backtrace)
-        @dimensions   = {}
-        configure_from_params(params)
-
-        @environment_variables = ENV.to_hash || {}
-      end
-
-      def context # rubocop:disable Metrics/MethodLength
-        c = {
-          application_name:  InfluxDB::Rails.configuration.application_name,
-          application_root:  InfluxDB::Rails.configuration.application_root,
-          framework:         InfluxDB::Rails.configuration.framework,
-          framework_version: InfluxDB::Rails.configuration.framework_version,
-          language:          "Ruby",
-          language_version:  "#{RUBY_VERSION}-p#{RUBY_PATCHLEVEL}",
-          custom_data:       @custom_data,
-        }
-
-        InfluxDB::Rails.configuration.add_custom_exception_data(self)
-        c
-      end
-
-      def dimensions
-        {
-          class:    @exception.class.to_s,
-          method:   "#{@controller}##{@action}",
-          filename: File.basename(@backtrace.lines.first.try(:file)),
-          server:   Socket.gethostname,
-          status:   "open",
-        }.merge(@dimensions)
-      end
-
-      def values
-        {
-          exception_message:   @exception.message,
-          exception_backtrace: JSON.generate(@backtrace.to_a),
-        }
-      end
-
-      def request_data
-        {
-          params:       @params,
-          session_data: @session_data,
-          controller:   @controller,
-          action:       @action,
-          request_url:  @request_url,
-          referer:      @referer,
-          remote_ip:    @remote_ip,
-          user_agent:   @user_agent,
-        }
-      end
-
-      private
-
-      def configure_from_params(params)
-        @params       = params[:params]
-        @session_data = params[:session_data] || {}
-        @current_user = params[:current_user]
-        @controller   = params[:controller]
-        @action       = params[:action]
-        @request_url  = params[:request_url]
-        @user_agent   = params[:user_agent]
-        @referer      = params[:referer]
-        @remote_ip    = params[:remote_ip]
-        @custom_data  = params[:custom_data] || {}
-      end
-    end
-  end
-end

data/lib/influxdb/rails/instrumentation.rb

@@ -1,34 +0,0 @@
-module InfluxDB
-  module Rails
-    module Instrumentation # rubocop:disable Style/Documentation
-      def benchmark_for_instrumentation # rubocop:disable Metrics/MethodLength
-        start = Time.now
-        yield
-
-        c = InfluxDB::Rails.configuration
-        return if c.ignore_current_environment?
-
-        InfluxDB::Rails.client.write_point \
-          c.series_name_for_instrumentation,
-          values: {
-            value: ((Time.now - start) * 1000).ceil,
-          },
-          tags:   configuration.tags_middleware.call(
-            method: "#{controller_name}##{action_name}",
-            server: Socket.gethostname
-          )
-      end
-
-      def self.included(base)
-        base.extend(ClassMethods)
-      end
-
-      module ClassMethods # rubocop:disable Style/Documentation
-        def instrument(methods = [])
-          methods = [methods] unless methods.is_a?(Array)
-          around_filter :benchmark_for_instrumentation, only: methods
-        end
-      end
-    end
-  end
-end

data/lib/influxdb/rails/logger.rb

@@ -1,16 +0,0 @@
-module InfluxDB
-  module Rails
-    module Logger # rubocop:disable Style/Documentation
-      PREFIX = "[InfluxDB::Rails] ".freeze
-
-      private
-
-      def log(level, message)
-        c = InfluxDB::Rails.configuration
-        return if level != :error && !c.debug?
-
-        c.logger&.send(level, PREFIX + message)
-      end
-    end
-  end
-end

data/lib/influxdb/rails/middleware/hijack_render_exception.rb

@@ -1,16 +0,0 @@
-module InfluxDB
-  module Rails
-    module Middleware
-      module HijackRenderException # rubocop:disable Style/Documentation
-        def render_exception(env, ex)
-          controller = env["action_controller.instance"] || env.controller_instance
-          request_data = controller.try(:influxdb_request_data) || {}
-          unless InfluxDB::Rails.configuration.ignore_user_agent?(request_data[:user_agent])
-            InfluxDB::Rails.report_exception_unless_ignorable(ex, request_data)
-          end
-          super
-        end
-      end
-    end
-  end
-end

data/lib/influxdb/rails/middleware/hijack_rescue_action_everywhere.rb

@@ -1,31 +0,0 @@
-module InfluxDB
-  module Rails
-    module Middleware
-      module HijackRescueActionEverywhere # rubocop:disable Style/Documentation
-        def self.included(base)
-          base.send(:alias_method_chain, :rescue_action_in_public, :influxdb)
-          base.send(:alias_method_chain, :rescue_action_locally, :influxdb)
-        end
-
-        private
-
-        def rescue_action_in_public_with_influxdb(ex)
-          handle_exception(ex)
-          rescue_action_in_public_without_influxdb(ex)
-        end
-
-        def rescue_action_locally_with_influxdb(ex)
-          handle_exception(ex)
-          rescue_action_locally_without_influxdb(ex)
-        end
-
-        def handle_exception(ex)
-          request_data = influxdb_request_data || {}
-          return if InfluxDB::Rails.configuration.ignore_user_agent?(request_data[:user_agent])
-
-          InfluxDB::Rails.report_exception_unless_ignorable(ex, request_data)
-        end
-      end
-    end
-  end
-end

data/lib/influxdb/rails/middleware/simple_subscriber.rb

@@ -1,46 +0,0 @@
-require "influxdb/rails/middleware/subscriber"
-
-module InfluxDB
-  module Rails
-    module Middleware
-      # Subscriber acts as base class for different *Subscriber classes,
-      # which are intended as ActiveSupport::Notifications.subscribe
-      # consumers.
-      class SimpleSubscriber < Subscriber
-        attr_reader :series_name
-
-        def initialize(configuration, series_name)
-          super(configuration)
-          @series_name = series_name
-        end
-
-        def call(_name, started, finished, _unique_id, payload)
-          return unless enabled?
-
-          begin
-            InfluxDB::Rails.client.write_point series_name,
-                                               values:    values(started, finished, payload),
-                                               tags:      tags(payload),
-                                               timestamp: timestamp(finished.utc)
-          rescue StandardError => e
-            log :error, "[InfluxDB::Rails] Unable to write points: #{e.message}"
-          end
-        end
-
-        private
-
-        def values(started, finished, _payload)
-          { value: ((finished - started) * 1000).ceil }
-        end
-
-        def timestamp(finished)
-          InfluxDB.convert_timestamp(finished.utc, configuration.time_precision)
-        end
-
-        def enabled?
-          super && series_name.present?
-        end
-      end
-    end
-  end
-end

data/lib/influxdb/rails/rack.rb

@@ -1,24 +0,0 @@
-module InfluxDB
-  module Rails
-    class Rack # rubocop:disable Style/Documentation
-      def initialize(app)
-        @app = app
-      end
-
-      def call(env)
-        dup._call(env)
-      end
-
-      def _call(env)
-        begin
-          response = @app.call(env)
-        rescue StandardError => e
-          InfluxDB::Rails.transmit_unless_ignorable(e, env)
-          raise(e)
-        end
-
-        response
-      end
-    end
-  end
-end

data/spec/controllers/widgets_controller_spec.rb

@@ -1,15 +0,0 @@
-require "spec_helper"
-
-RSpec.describe WidgetsController, type: :controller do
-  describe "#new" do
-    it "should raise an exception" do
-      expect { get :new }.to raise_error(ZeroDivisionError)
-    end
-  end
-
-  describe "#index" do
-    it "should not raise an exception" do
-      expect { get :index }.to_not raise_error
-    end
-  end
-end

data/spec/integration/exceptions_spec.rb

@@ -1,37 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + "/integration_helper")
-
-RSpec.describe "exception handling", type: :request do
-  before do
-    InfluxDB::Rails.configure do |config|
-      config.ignored_environments = %w[development]
-      config.instrumentation_enabled = false
-    end
-  end
-
-  describe "in an action that raises an exception" do
-    it "should add an exception to the queue" do
-      expect(InfluxDB::Rails.client).to receive(:write_point)
-      get "/widgets/new"
-    end
-  end
-
-  describe "in an action that does not raise an exception" do
-    it "should not add anything to the queue" do
-      expect(InfluxDB::Rails.client).not_to receive(:write_point)
-      get "/widgets"
-    end
-  end
-
-  describe "for an ignored user agent" do
-    it "should not make an HTTP call to the API" do
-      expect(InfluxDB::Rails.client).not_to receive(:write_point)
-
-      # note: GoogleBot is ignored by default
-      if Rails::VERSION::MAJOR >= 5
-        get "/widgets/new", headers: { "HTTP_USER_AGENT" => "GoogleBot/2.1" }
-      else
-        get "/widgets/new", {}, "HTTP_USER_AGENT" => "GoogleBot/2.1"
-      end
-    end
-  end
-end