airbrake-ruby 5.0.0.rc.1 → 5.0.0.rc.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f8307b88b3240291a93fa8e8590997aca70032e424edff8f3ef2cd8c7220c4b3
-  data.tar.gz: 53a68e1f1bab4158d773593ad201dce948a0df60c0fd1b80a27451a96dc5596c
+  metadata.gz: 36bf29e9b0f7ccd10afe2fdc47ec434c02941e8ad0f82e7806280b731db96bcf
+  data.tar.gz: bcc5563ebce4151570184bf9ed9248eb21684fbc4cf5911fbea687f47e4587e4
 SHA512:
-  metadata.gz: fa9bcb8eac54050e2065c850ce7056a219c7e6a3992bac5b2eeab36d95a39d861bfbd4a9e81ba6d377b55e32679723f013cbcbc213ab6f6a29715f544a719259
-  data.tar.gz: '08fdf62186815e85696acd66641f60bfeca556c2a3e30c0b7d7c49c3f02bdd208d795aceee9af65a4683773580682f8efa96a61434a716118a8a85346432d41f'
+  metadata.gz: edf639bcf65ab8686114b93c920498605f6cfabce2ef6a6d4c364bd507076c0da80bd26358ce2414dfcf65c24b13f6665e0718a453d0e0e50f483077c2237e77
+  data.tar.gz: 9967e9fa9ddef2d7a3be67aa0fc1cc44eca5a0c89945b0fa69bf4510733e827f164a70c7a12e36426d4a5b6f239fdd072a64de886aca42d93355ad18b5d96b66

data/lib/airbrake-ruby/notice.rb

@@ -4,19 +4,12 @@ module Airbrake
   #
   # @since v1.0.0
   class Notice
-    # @return [Hash{Symbol=>String}] the information about the notifier library
-    NOTIFIER = {
-      name: 'airbrake-ruby'.freeze,
-      version: Airbrake::AIRBRAKE_RUBY_VERSION,
-      url: 'https://github.com/airbrake/airbrake-ruby'.freeze,
-    }.freeze
-
     # @return [Hash{Symbol=>String,Hash}] the information to be displayed in the
     #   Context tab in the dashboard
     CONTEXT = {
       os: RUBY_PLATFORM,
       language: "#{RUBY_ENGINE}/#{RUBY_VERSION}".freeze,
-      notifier: NOTIFIER,
+      notifier: Airbrake::NOTIFIER_INFO,
     }.freeze
 
     # @return [Integer] the maxium size of the JSON payload in bytes

data/lib/airbrake-ruby/remote_settings.rb

@@ -24,6 +24,15 @@ module Airbrake
       '../../config/config.json',
     ).freeze
 
+    # @return [Hash{Symbol=>String}] metadata to be attached to every GET
+    #   request
+    QUERY_PARAMS = URI.encode_www_form(
+      notifier_name: Airbrake::NOTIFIER_INFO[:name],
+      notifier_version: Airbrake::NOTIFIER_INFO[:version],
+      os: RUBY_PLATFORM,
+      language: "#{RUBY_ENGINE}/#{RUBY_VERSION}".freeze,
+    ).freeze
+
     # Polls remote config of the given project.
     #
     # @param [Integer] project_id
@@ -84,7 +93,7 @@ module Airbrake
     def fetch_config
       response = nil
       begin
-        response = Net::HTTP.get(URI(@data.config_route))
+        response = Net::HTTP.get(build_config_uri)
       rescue StandardError => ex
         logger.error(ex)
         return {}
@@ -108,6 +117,12 @@ module Airbrake
       json
     end
 
+    def build_config_uri
+      uri = URI(@data.config_route)
+      uri.query = QUERY_PARAMS
+      uri
+    end
+
     def load_config
       config_dir = File.dirname(CONFIG_DUMP_PATH)
       Dir.mkdir(config_dir) unless File.directory?(config_dir)

data/lib/airbrake-ruby/remote_settings/settings_data.rb

@@ -22,7 +22,7 @@ module Airbrake
 
       # @return [String] what URL to poll
       CONFIG_ROUTE_PATTERN =
-        'https://%<bucket>s.s3.amazonaws.com/' \
+        'https://v1-%<bucket>s.s3.amazonaws.com/' \
         "#{API_VER}/config/%<project_id>s/config.json".freeze
 
       # @return [Hash{Symbol=>String}] the hash of all supported settings where

data/lib/airbrake-ruby/sync_sender.rb

@@ -79,7 +79,7 @@ module Airbrake
       req['Authorization'] = "Bearer #{@config.project_key}"
       req['Content-Type'] = CONTENT_TYPE
       req['User-Agent'] =
-        "#{Airbrake::Notice::NOTIFIER[:name]}/#{Airbrake::AIRBRAKE_RUBY_VERSION}" \
+        "#{Airbrake::NOTIFIER_INFO[:name]}/#{Airbrake::AIRBRAKE_RUBY_VERSION}" \
         " Ruby/#{RUBY_VERSION}"
 
       req

data/lib/airbrake-ruby/version.rb

@@ -2,5 +2,15 @@
 # More information: http://semver.org/
 module Airbrake
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '5.0.0.rc.1'.freeze
+  # @api public
+  AIRBRAKE_RUBY_VERSION = '5.0.0.rc.2'.freeze
+
+  # @return [Hash{Symbol=>String}] the information about the notifier library
+  # @since ?.?.?
+  # @api public
+  NOTIFIER_INFO = {
+    name: 'airbrake-ruby'.freeze,
+    version: Airbrake::AIRBRAKE_RUBY_VERSION,
+    url: 'https://github.com/airbrake/airbrake-ruby'.freeze,
+  }.freeze
 end

data/spec/remote_settings/settings_data_spec.rb

@@ -79,7 +79,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
       it "returns the default pathname" do
         expect(described_class.new(project_id, data).config_route).to eq(
-          'https://staging-notifier-configs.s3.amazonaws.com/' \
+          'https://v1-staging-notifier-configs.s3.amazonaws.com/' \
           "2020-06-18/config/#{project_id}/config.json",
         )
       end

data/spec/remote_settings_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe Airbrake::RemoteSettings do
   let(:project_id) { 123 }
 
   let(:endpoint) do
-    "https://staging-notifier-configs.s3.amazonaws.com/2020-06-18/config/" \
+    "https://v1-staging-notifier-configs.s3.amazonaws.com/2020-06-18/config/" \
     "#{project_id}/config.json"
   end
 
@@ -25,9 +25,12 @@ RSpec.describe Airbrake::RemoteSettings do
   let(:config_path) { described_class::CONFIG_DUMP_PATH }
   let(:config_dir) { File.dirname(config_path) }
 
-  before do
-    stub_request(:get, endpoint).to_return(status: 200, body: body.to_json)
+  let!(:stub) do
+    stub_request(:get, Regexp.new(endpoint))
+      .to_return(status: 200, body: body.to_json)
+  end
 
+  before do
     # Do not create config dumps on disk.
     allow(Dir).to receive(:mkdir).with(config_dir)
     allow(File).to receive(:write).with(config_path, anything)
@@ -52,7 +55,7 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.2)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
       end
 
       it "yields the config to the block twice" do
@@ -63,7 +66,7 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.2)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
       end
 
       context "when config loading fails" do
@@ -77,7 +80,7 @@ RSpec.describe Airbrake::RemoteSettings do
           sleep(0.2)
           remote_settings.stop_polling
 
-          expect(a_request(:get, endpoint)).to have_been_made.once
+          expect(stub).to have_been_requested.once
         end
       end
     end
@@ -88,7 +91,18 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.1)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.at_least_once
+        expect(stub).to have_been_requested.at_least_once
+      end
+
+      it "sends params about the environment with the request" do
+        remote_settings = described_class.poll(project_id) {}
+        sleep(0.1)
+        remote_settings.stop_polling
+
+        stub_with_query_params = stub.with(
+          query: URI.decode_www_form(described_class::QUERY_PARAMS).to_h,
+        )
+        expect(stub_with_query_params).to have_been_requested.at_least_once
       end
 
       it "fetches remote settings" do
@@ -118,7 +132,7 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.1)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).not_to have_been_made
+        expect(stub).not_to have_been_requested
         expect(settings.interval).to eq(600)
       end
     end
@@ -136,14 +150,15 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.1)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
         expect(settings.interval).to eq(600)
       end
     end
 
     context "when API returns an XML response" do
-      before do
-        stub_request(:get, endpoint).to_return(status: 200, body: '<?xml ...')
+      let!(:stub) do
+        stub_request(:get, Regexp.new(endpoint))
+          .to_return(status: 200, body: '<?xml ...')
       end
 
       it "doesn't update settings data" do
@@ -154,20 +169,23 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.1)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
         expect(settings.interval).to eq(600)
       end
     end
 
     context "when a config route is specified in the returned data" do
-      let(:new_endpoint) { 'http://example.com' }
+      let(:new_endpoint) do
+        "http://example.com"
+      end
 
       let(:body) do
         { 'config_route' => new_endpoint, 'poll_sec' => 0.1 }
       end
 
-      before do
-        stub_request(:get, new_endpoint).to_return(status: 200, body: body.to_json)
+      let!(:new_stub) do
+        stub_request(:get, Regexp.new(new_endpoint))
+          .to_return(status: 200, body: body.to_json)
       end
 
       it "makes the next request to the specified config route" do
@@ -176,8 +194,8 @@ RSpec.describe Airbrake::RemoteSettings do
 
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
-        expect(a_request(:get, new_endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
+        expect(new_stub).to have_been_requested.once
       end
     end
   end
@@ -191,7 +209,7 @@ RSpec.describe Airbrake::RemoteSettings do
       sleep(0.2)
       remote_settings.stop_polling
 
-      expect(a_request(:get, endpoint)).to have_been_made.once
+      expect(stub).to have_been_requested.once
     end
 
     context "when config dumping fails" do
@@ -205,7 +223,7 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.2)
         remote_settings.stop_polling
 
-        expect(a_request(:get, endpoint)).to have_been_made.once
+        expect(stub).to have_been_requested.once
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 5.0.0.rc.1
+  version: 5.0.0.rc.2
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-13 00:00:00.000000000 Z
+date: 2020-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree3
@@ -152,7 +152,7 @@ homepage: https://airbrake.io
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -168,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Ruby notifier for https://airbrake.io
 test_files: