Revert "Custom backend"

This reverts commit 1bda33d.

lib/honeybadger/backend.rb

@@ -5,7 +5,6 @@
 require 'honeybadger/backend/test'
 require 'honeybadger/backend/null'
 require 'honeybadger/backend/debug'
-require 'honeybadger/backend/events'
 
 module Honeybadger
   # @api private
@@ -17,8 +16,7 @@ def self.mapping
         server: Server,
         test: Test,
         null: Null,
-        debug: Debug,
-        events: Events,
+        debug: Debug
       }.freeze
     end
 

lib/honeybadger/backend/server.rb

@@ -12,6 +12,7 @@ class Server < Base
       ENDPOINTS = {
         notices: '/v1/notices'.freeze,
         deploys: '/v1/deploys'.freeze,
+        logs: '/v1/events'.freeze,
       }.freeze
 
       CHECK_IN_ENDPOINT = '/v1/check_in'.freeze

lib/honeybadger/config.rb

@@ -151,11 +151,6 @@ def backend
       @backend
     end
 
-    def logs_backend
-      init_logs_backend! unless @logs_backend
-      @logs_backend
-    end
-
     def backend=(backend)
       set(:backend, backend)
       @backend = nil
@@ -338,18 +333,6 @@ def init_backend!
       @backend = default_backend
     end
 
-    def init_logs_backend!
-      if self[:logs_backend].is_a?(String) || self[:logs_backend].is_a?(Symbol)
-        @logs_backend = Backend.for(self[:logs_backend].to_sym).new(self)
-        return
-      end
-
-      if ruby[:logs_backend].respond_to?(:notify)
-        @logs_backend = ruby[:logs_backend]
-        return
-      end
-    end
-
     def build_stdout_logger
       logger = ::Logger.new($stdout)
       logger.formatter = lambda do |severity, datetime, progname, msg|

lib/honeybadger/config/defaults.rb

@@ -71,11 +71,6 @@ class Boolean; end
         default: nil,
         type: String
       },
-      logs_backend: {
-        description: 'An alternate backend to use for the logger feature.',
-        default: :events,
-        type: String
-      },
       debug: {
         description: 'Enables debug logging.',
         default: false,

lib/honeybadger/logger.rb

@@ -76,7 +76,6 @@ class HttpAppender < SemanticLogger::Subscriber
 
       def initialize(*)
         super
-        @backend = Honeybadger.config.logs_backend
         @retry_queue = []
       end
 
@@ -86,7 +85,8 @@ def default_formatter
 
       def batch(logs)
         payload = logs.map { |log| formatter.call(log, self) }.join("\n")
-        response = @backend.notify(:logs, payload)
+        def payload.to_json; self; end # The Server backend calls to_json
+        response = Honeybadger.config.backend.notify(:logs, payload)
 
         if response.success?
           retry_previous_failed_requests
@@ -104,7 +104,7 @@ def retry_previous_failed_requests
         can_send = true
         until @retry_queue.empty? || !can_send
           payload = @retry_queue.shift
-          response = @backend.notify(:logs, payload)
+          response = Honeybadger.config.backend.notify(:logs, payload)
           if !response.success?
             @retry_queue.unshift(payload)
             can_send = false

spec/unit/honeybadger/logger_spec.rb

@@ -4,7 +4,7 @@
   before do
     Honeybadger.logger.instance_variable_set(:@appender, nil)
     Honeybadger::Backend::Test.notifications[:logs] = []
-    Honeybadger.config[:logs_backend] = :test
+    Honeybadger.config[:backend] = :test
   end
 
   after do
@@ -54,7 +54,10 @@ def wait_for_other_thread(wait = 0.05)
     Honeybadger.config[:"features.logger.batch_size"] = 200
     Honeybadger.config[:"features.logger.batch_interval"] = 0.001
 
-    expect(Honeybadger.config.logs_backend).to receive(:notify).twice do
+    backend = Honeybadger::Backend::Test.new(Honeybadger.config)
+    Honeybadger.config[:backend] = backend
+
+    expect(backend).to receive(:notify).twice do
       Honeybadger::Backend::Null::StubbedResponse.new(successful: false)
     end
     # Four messages across three batches, with the first two batches failing
@@ -66,7 +69,7 @@ def wait_for_other_thread(wait = 0.05)
 
     expect(received_batches.size).to eq(0)
 
-    expect(Honeybadger.config.logs_backend).to receive(:notify).exactly(3).times.and_call_original
+    expect(backend).to receive(:notify).exactly(3).times.and_call_original
     wait_for_other_thread { Honeybadger.logger.error("Fourth - succeeds") }
 
     expect(received_batches.size).to eq(3)