Use response[:status] instead of the less reliable response[:response…

…_headers]['status']
nsanta · Dec 25, 2010 · b115038 · b115038
1 parent 7ba01a0
commit b115038
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/faraday/raise_http_4xx.rb b/lib/faraday/raise_http_4xx.rb
@@ -31,7 +31,7 @@ def initialize(app)
     private
 
     def self.error_message(response)
-      "#{response[:method].to_s.upcase} #{response[:url].to_s}: #{response[:response_headers]['status']}#{error_body(response[:body])}"
+      "#{response[:method].to_s.upcase} #{response[:url].to_s}: #{response[:status]}#{error_body(response[:body])}"
     end
 
     def self.error_body(body)

diff --git a/lib/faraday/raise_http_5xx.rb b/lib/faraday/raise_http_5xx.rb
@@ -25,7 +25,7 @@ def initialize(app)
     private
 
     def self.error_message(response, body=nil)
-      "#{response[:method].to_s.upcase} #{response[:url].to_s}: #{response[:response_headers]['status']}:#{(' ' + body) if body} Check http://status.twitter.com/ for updates on the status of the Twitter service."
+      "#{response[:method].to_s.upcase} #{response[:url].to_s}: #{[response[:status].to_s + ':', body].compact.join(' ')} Check http://status.twitter.com/ for updates on the status of the Twitter service."
     end
   end
 end