WIP

Author: Steve Kirkland

CHANGELOG.md

@@ -1,6 +1,13 @@
 Changelog
 =========
 
+## v6.28.0 (** June 2024)
+
+### Enhancements
+
+* Set default endpoints based on API key
+| [#835](https://github.com/bugsnag/bugsnag-ruby/pull/835)
+
 ## v6.27.1 (18 June 2024)
 
 ### Fixes

lib/bugsnag.rb

@@ -54,6 +54,8 @@ class << self
     def configure(validate_api_key=true)
       yield(configuration) if block_given?
 
+      configuration.set_default_endpoints
+
       # Create the session tracker if sessions are enabled to avoid the overhead
       # of creating it on the first request. We skip this if we're not validating
       # the API key as we use this internally before the user's configure block
@@ -522,8 +524,12 @@ def check_key_valid
     # If only a notify_endpoint has been set, session tracking will be disabled
     # If only a session_endpoint has been set, and ArgumentError will be raised
     def check_endpoint_setup
-      notify_set = configuration.notify_endpoint && configuration.notify_endpoint != Bugsnag::Configuration::DEFAULT_NOTIFY_ENDPOINT
-      session_set = configuration.session_endpoint && configuration.session_endpoint != Bugsnag::Configuration::DEFAULT_SESSION_ENDPOINT
+      notify_set = configuration.notify_endpoint &&
+        configuration.notify_endpoint != Bugsnag::Configuration::DEFAULT_NOTIFY_ENDPOINT &&
+        configuration.notify_endpoint != Bugsnag::Configuration::HUB_NOTIFY
+      session_set = configuration.session_endpoint &&
+        configuration.session_endpoint != Bugsnag::Configuration::DEFAULT_SESSION_ENDPOINT &&
+        configuration.session_endpoint != Bugsnag::Configuration::HUB_SESSION
       if notify_set && !session_set
         configuration.warn("The session endpoint has not been set, all further session capturing will be disabled")
         configuration.disable_sessions

lib/bugsnag/configuration.rb

@@ -199,7 +199,9 @@ class Configuration
 
     DEFAULT_NOTIFY_ENDPOINT = "https://notify.bugsnag.com"
     DEFAULT_SESSION_ENDPOINT = "https://sessions.bugsnag.com"
-    DEFAULT_ENDPOINT = DEFAULT_NOTIFY_ENDPOINT
+    HUB_NOTIFY = "https://notify.insighthub.smartbear.com"
+    HUB_SESSION = "https://sessions.insighthub.smartbear.com"
+    HUB_PREFIX = "00000"
 
     DEFAULT_META_DATA_FILTERS = [
       /authorization/i,
@@ -249,7 +251,7 @@ def initialize
       # to avoid infinite recursion when creating breadcrumb buffer
       @max_breadcrumbs = DEFAULT_MAX_BREADCRUMBS
 
-      @endpoints = EndpointConfiguration.new(DEFAULT_NOTIFY_ENDPOINT, DEFAULT_SESSION_ENDPOINT)
+      @endpoints = EndpointConfiguration.new(nil, nil)
 
       @enable_events = true
       @enable_sessions = true
@@ -540,6 +542,22 @@ def session_endpoint=(new_session_endpoint)
       set_endpoints(notify_endpoint, new_session_endpoint) # Pass the existing notify_endpoint through so it doesn't get overwritten
     end
 
+    ##
+    # Sets the notification and session endpoints to default values if neither have been set
+    #
+    def set_default_endpoints
+
+      puts @endpoints.inspect
+
+      if @endpoints.notify.nil? && @endpoints.sessions.nil?
+        if self.hub_api_key?
+          self.endpoints = EndpointConfiguration.new(HUB_NOTIFY, HUB_SESSION)
+        else
+          self.endpoints = EndpointConfiguration.new(DEFAULT_NOTIFY_ENDPOINT, DEFAULT_SESSION_ENDPOINT)
+        end
+      end
+    end
+
     ##
     # Sets the notification and session endpoints
     #
@@ -753,5 +771,9 @@ def default_hostname
       # Send the heroku dyno name instead of hostname if available
       ENV["DYNO"] || Socket.gethostname;
     end
+
+    def hub_api_key?
+      @api_key && @api_key.start_with?(HUB_PREFIX)
+    end
   end
 end

spec/configuration_spec.rb

@@ -107,8 +107,8 @@
 
   describe "endpoint configuration" do
     describe "#notify_endpoint" do
-      it "defaults to DEFAULT_NOTIFY_ENDPOINT" do
-        expect(subject.notify_endpoint).to eq(Bugsnag::Configuration::DEFAULT_NOTIFY_ENDPOINT)
+      it "defaults to nil" do
+        expect(subject.notify_endpoint).to eq(nil)
       end
 
       it "is readonly" do
@@ -121,11 +121,15 @@
     end
 
     describe "#session_endpoint" do
-      it "defaults to DEFAULT_SESSION_ENDPOINT" do
-        expect(subject.session_endpoint).to eq(Bugsnag::Configuration::DEFAULT_SESSION_ENDPOINT)
+      it "defaults to nil" do
+        expect(subject.session_endpoint).to eq(nil)
       end
     end
 
+    describe "#set_default_endpoints" do
+
+    end
+
     describe "#endpoint=" do
       let(:custom_notify_endpoint) { "My custom notify endpoint" }
       let(:session_endpoint) { "My session endpoint" }
@@ -171,11 +175,11 @@
     end
 
     describe "#endpoints" do
-      it "defaults to 'DEFAULT_NOTIFY_ENDPOINT' & 'DEFAULT_SESSION_ENDPOINT'" do
+      it "defaults to nil & nil" do
         config = Bugsnag::Configuration.new
 
-        expect(config.endpoints.notify).to eq(Bugsnag::Configuration::DEFAULT_NOTIFY_ENDPOINT)
-        expect(config.endpoints.sessions).to eq(Bugsnag::Configuration::DEFAULT_SESSION_ENDPOINT)
+        expect(config.endpoints.notify).to eq(nil)
+        expect(config.endpoints.sessions).to eq(nil)
       end
     end
 
@@ -203,7 +207,7 @@
         expect(config.enable_sessions).to be(false)
       end
 
-      # TODO: this behaviour exists for backwards compatibilitiy
+      # TODO: this behaviour exists for backwards compatibility
       #       ideally we should not send events in this case
       it "warns and disables sessions if only notify URL is given" do
         config = Bugsnag::Configuration.new
@@ -427,6 +431,24 @@ def output_lines
             '[Bugsnag] WARN: No valid API key has been set, notifications will not be sent'
           )
         end
+
+        it "uses the default endpoints for non-hub API keys" do
+          puts "test 1"
+          Bugsnag.configure do |config|
+            config.api_key = '00002472bd130ac0ab0f52715bbdc600'
+          end
+          expect(subject.endpoints.notify).to eq(Bugsnag::Configuration::DEFAULT_NOTIFY_ENDPOINT)
+          expect(subject.endpoints.sessions).to eq(Bugsnag::Configuration::DEFAULT_SESSION_ENDPOINT)
+        end
+
+        it "uses the hub endpoints for hub API keys" do
+          puts "test 2"
+          Bugsnag.configure do |config|
+            config.api_key = '00000472bd130ac0ab0f52715bbdc600'
+          end
+          expect(subject.endpoints.notify).to eq(Bugsnag::Configuration::HUB_NOTIFY)
+          expect(subject.endpoints.sessions).to eq(Bugsnag::Configuration::HUB_SESSION)
+        end
       end
     end