jeff 0.5.0 → 0.6.0

data/README.md

@@ -9,6 +9,6 @@
 
 [stat]: https://secure.travis-ci.org/papercavalier/jeff.png
 [trav]: http://travis-ci.org/papercavalier/jeff
+[sign]: http://docs.amazonwebservices.com/general/latest/gr/signature-version-2.html
 [aws]:  http://aws.amazon.com/
 [jeff]: http://f.cl.ly/items/0a3R3J0k1R2f423k1q2l/jeff.jpg
-[sign]: http://docs.amazonwebservices.com/general/latest/gr/signature-version-2.html

data/lib/jeff.rb

@@ -8,9 +8,9 @@ require 'jeff/version'
 
 # Mixes in Amazon Web Services (AWS) client behaviour.
 module Jeff
-  MissingEndpoint = Class.new ArgumentError
-  MissingKey      = Class.new ArgumentError
-  MissingSecret   = Class.new ArgumentError
+  MissingEndpoint = Class.new(ArgumentError)
+  MissingKey      = Class.new(ArgumentError)
+  MissingSecret   = Class.new(ArgumentError)
 
   UNRESERVED = /([^\w.~-]+)/
 
@@ -31,7 +31,7 @@ module Jeff
                  'Timestamp'        => -> { Time.now.utc.iso8601 }
   end
 
-  # Internal: Builds a sorted query.
+  # Internal: Build a sorted query.
   #
   # hsh - A hash of query parameters specific to the request.
   #
@@ -40,7 +40,7 @@ module Jeff
     params
       .merge(hsh)
       .sort
-      .map { |k, v| "#{k}=#{ escape v }" }
+      .map { |k, v| "#{k}=#{ escape(v) }" }
       .join('&')
   end
 
@@ -102,27 +102,30 @@ module Jeff
     eval <<-DEF
       def #{method}(opts = {})
         opts.update :method => :#{method}
-        if opts[:body]
-          opts[:headers] ||= {}
-          opts[:headers].update 'Content-MD5' => calculate_md5(opts[:body])
-        end
-
-        connection.request(sign(opts))
+        connection.request(build_options(opts))
       end
     DEF
   end
 
   private
 
+  def build_options(opts)
+    opts[:headers] ||= {}
+    opts[:headers].update('Host' => connection_host)
+
+    if opts[:body]
+      opts[:headers].update('Content-MD5' => calculate_md5(opts[:body]))
+    end
+
+    sign(opts)
+  end
+
   def calculate_md5(body)
     Base64.encode64(Digest::MD5.digest(body)).strip
   end
 
   def connection_host
-    [
-      connection.data[:host],
-      connection.data[:port]
-    ].join ':'
+    connection.data[:host]
   end
 
   def connection_path
@@ -143,7 +146,7 @@ module Jeff
       connection_host,
       opts[:path] || connection_path,
       query
-    ].join "\n"
+    ].join("\n")
     signature = secret.sign(string_to_sign)
 
     opts.update query: [

data/lib/jeff/version.rb

@@ -1,3 +1,3 @@
 module Jeff
-  VERSION = '0.5.0'
+  VERSION = '0.6.0'
 end

metadata

@@ -2,14 +2,14 @@
 name: jeff
 version: !ruby/object:Gem::Version
   prerelease: 
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Hakan Ensari
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-24 00:00:00.000000000 Z
+date: 2013-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   version_requirements: !ruby/object:Gem::Requirement