airbrake-ruby 5.0.0 → 5.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e98183a856071ce3d105c36adc9aa22a8ee7071b93958fd70fb673d2735b3977
-  data.tar.gz: 80464931a1362c4d845ffee737ed7aa1e46693136023b1f08006c49de98e62eb
+  metadata.gz: 773b982037081f5d6dfb045378a35b2854a251d9e238cd58dd0b1216c75e86c5
+  data.tar.gz: a7177673edea3b8f05055683972b918bf43255ad68fc21688317ec5ed56dbb7b
 SHA512:
-  metadata.gz: 8c32fc631e3af6348a871094dde31452d4a6fdef834596933e179d2acdbbd4935751715007fa7c4dcfce3639ebbd55f61b15f06428015647e4cbcac590ef7aaf
-  data.tar.gz: 23968940c6f1dfb500be5ab479c1d524cdc9f80e40808b4ff08d7f4b03c97baad603d97504c58a5300ddfa7ec9b1a4b5bd44a24a6a4a15776a82778ef2a0c324
+  metadata.gz: 6e377bdd85f7aa912cc9341834f9413ec5a89b71bbe2acf561118be97a911baef6c7cc210fa2430e286d135ce829ca7009110180c5938ac7017a403d7be239de
+  data.tar.gz: b9e6812ed5d2734c7fd3ffd83b14e5d366594366c5460244e43d13375ede211c56d4e1a3b4c79b613de0d480493d9950a833670bb2f59161f1d0db051936b0a1

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

@@ -58,17 +58,18 @@ module Airbrake
       # @param [String] remote_config_host
       # @return [String] where the config is stored on S3.
       def config_route(remote_config_host)
-        if !@data.key?('config_route') || !@data['config_route']
+        if @data.key?('config_route') &&
+           @data['config_route'] && !@data['config_route'].empty?
           return format(
             CONFIG_ROUTE_PATTERN,
-            host: remote_config_host.chomp('/'),
+            host: @data['config_route'].chomp('/'),
             project_id: @project_id,
           )
         end
 
         format(
           CONFIG_ROUTE_PATTERN,
-          host: @data['config_route'].chomp('/'),
+          host: remote_config_host.chomp('/'),
           project_id: @project_id,
         )
       end

data/lib/airbrake-ruby/version.rb

@@ -3,7 +3,7 @@
 module Airbrake
   # @return [String] the library version
   # @api public
-  AIRBRAKE_RUBY_VERSION = '5.0.0'.freeze
+  AIRBRAKE_RUBY_VERSION = '5.0.1'.freeze
 
   # @return [Hash{Symbol=>String}] the information about the notifier library
   # @since 5.0.0

data/spec/remote_settings/settings_data_spec.rb

@@ -102,7 +102,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
       context "and when the remote host doesn't with a slash" do
         let(:host) { 'http://example.com' }
 
-        it "returns the route with the host" do
+        it "returns the route with the given host" do
           expect(described_class.new(project_id, {}).config_route(host)).to eq(
             "http://example.com/2020-06-18/config/#{project_id}/config.json",
           )
@@ -122,6 +122,19 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
         )
       end
     end
+
+    context "when the given remote host in the remote config is an empty string" do
+      let(:data) do
+        { 'config_route' => '' }
+      end
+
+      it "returns the route with the default instead" do
+        expect(described_class.new(project_id, data).config_route(host)).to eq(
+          'https://v1-production-notifier-configs.s3.amazonaws.com/' \
+          "2020-06-18/config/#{project_id}/config.json",
+        )
+      end
+    end
   end
 
   describe "#error_notifications?" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 5.0.1
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.