Fixing/updating the hoptoad backend

The previous version of the hoptoad back end no longer worked. This is due to Hoptoad changing their API somewhere along the way.
I've updated the code here to simply use the hoptoad_notifier gem that folks are likely already using in their apps. This puts the
maintenance and configuration responsibility into that gem rather than here in resque.

Author: raykrueger

lib/resque/failure/hoptoad.rb

@@ -1,6 +1,4 @@
-require 'net/https'
-require 'builder'
-require 'uri'
+require 'hoptoad_notifier'
 
 module Resque
   module Failure
@@ -10,130 +8,33 @@ module Failure
     #
     #   require 'resque/failure/hoptoad'
     #
-    #   Resque::Failure::Hoptoad.configure do |config|
-    #     config.api_key = 'blah'
-    #     config.secure = true
+    #   Resque::Failure::Multiple.classes = [Resque::Failure::Redis, Resque::Failure::Hoptoad]
+    #   Resque::Failure.backend = Resque::Failure::Multiple
     #
-    #     # optional proxy support
-    #     config.proxy_host = 'x.y.z.t'
-    #     config.proxy_port = 8080
-    #
-    #     # server env support, defaults to RAILS_ENV or RACK_ENV
-    #     config.server_environment = "test"
-    #   end
+    # Once you've configured resque to use the Hoptoad failure backend,
+    # you'll want to setup an initializer to configure the Hoptoad.
+    # 
+    # HoptoadNotifier.configure do |config|
+    #   config.api_key = 'your_key_here'
+    # end
+    # For more information see https://github.com/thoughtbot/hoptoad_notifier 
     class Hoptoad < Base
-      # From the hoptoad plugin
-      INPUT_FORMAT = /^([^:]+):(\d+)(?::in `([^']+)')?$/
-
-      class << self
-        attr_accessor :secure, :api_key
-        attr_accessor :proxy_host, :proxy_port, :proxy_user, :proxy_pass
-        attr_accessor :server_environment
-        attr_accessor :host, :port
-        attr_accessor :http_read_timeout, :http_open_timeout
-      end
 
       def self.count
         # We can't get the total # of errors from Hoptoad so we fake it
         # by asking Resque how many errors it has seen.
         Stat[:failed]
       end
 
-      def self.configure
-        yield self
-        Resque::Failure.backend = self
-      end
-
       def save
-        http = use_ssl? ? :https : :http
-        host = self.class.host || 'hoptoadapp.com'
-        port = self.class.port
-        url = URI.parse("#{http}://#{host}:#{port}/notifier_api/v2/notices/")
-
-        request = Net::HTTP::Proxy self.class.proxy_host, self.class.proxy_port,
-                                   self.class.proxy_user, self.class.proxy_pass
-        http = request.new(url.host, url.port)
-        headers = {
-          'Content-type' => 'text/xml',
-          'Accept' => 'text/xml, application/xml'
-        }
-
-        http.read_timeout = self.class.http_read_timeout || 5 # seconds
-        http.open_timeout = self.class.http_open_timeout || 2 # seconds
-
-        http.use_ssl = use_ssl?
-
-        begin
-          response = http.post(url.path, xml, headers)
-        rescue TimeoutError => e
-          log "Timeout while contacting the Hoptoad server."
-        end
-
-        case response
-        when Net::HTTPSuccess then
-          log "Hoptoad Success: #{response.class}"
-        else
-          body = response.body if response.respond_to? :body
-          log "Hoptoad Failure: #{response.class}\n#{body}"
-        end
+        HoptoadNotifier.notify(exception,
+          :parameters => {
+            :payload_class => payload['class'].to_s,
+            :payload_args => payload['args'].inspect
+          }
+        )
       end
 
-      def xml
-        x = Builder::XmlMarkup.new
-        x.instruct!
-        x.notice :version=>"2.0" do
-          x.tag! "api-key", api_key
-          x.notifier do
-            x.name "Resqueue"
-            x.version "0.1"
-            x.url "http://github.com/defunkt/resque"
-          end
-          x.error do
-            x.tag! "class", exception.class.name
-            x.message "#{exception.class.name}: #{exception.message}"
-            x.backtrace do
-              fill_in_backtrace_lines(x)
-            end
-          end
-          x.request do
-            x.url queue.to_s
-            x.component worker.to_s
-            x.params do
-              x.var :key=>"payload_class" do
-                x.text! payload["class"].to_s
-              end
-              x.var :key=>"payload_args" do
-                x.text! payload["args"].to_s
-              end
-            end
-          end
-          x.tag!("server-environment") do
-            x.tag!("project-root", "RAILS_ROOT")
-            x.tag!("environment-name",server_environment)
-          end
-
-        end
-      end
-
-      def fill_in_backtrace_lines(x)
-        Array(exception.backtrace).each do |unparsed_line|
-          _, file, number, method = unparsed_line.match(INPUT_FORMAT).to_a
-          x.line :file => file,:number => number
-        end
-      end
-
-      def use_ssl?
-        self.class.secure
-      end
-
-      def api_key
-        self.class.api_key
-      end
-
-      def server_environment
-        return self.class.server_environment if self.class.server_environment
-        defined?(RAILS_ENV) ? RAILS_ENV : (ENV['RACK_ENV'] || 'development')
-      end
     end
   end
 end