rack-cachely 0.2.0 → 0.2.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rack-cachely (0.1.0)
+    rack-cachely (0.2.1)
       rack
 
 GEM

data/lib/rack-cachely/key.rb

@@ -25,9 +25,9 @@ module Rack
           parts << @request.script_name
           parts << @request.path_info
 
-          if qs = query_string
+          if query_string && query_string != ""
             parts << "?"
-            parts << qs
+            parts << query_string
           end
 
           parts.join

data/lib/rack-cachely/store.rb

@@ -9,7 +9,7 @@ module Rack
 
       def get(key)
         remote do
-          url = "#{config.cachely_url}?url=#{CGI.escape(key.to_s)}"
+          url = "#{config.cachely_url}?_version=#{Rack::Cachely::VERSION}&url=#{CGI.escape(key.to_s)}"
           uri = URI(url)
           http = Net::HTTP.new(uri.host, uri.port)
           request = Net::HTTP::Get.new(uri.request_uri)
@@ -43,7 +43,8 @@ module Rack
             "document[url]" => key,
             "document[status]" => rack[0].to_i,
             "document[body]" => body,
-            "document[age]" => options[:age] || 0
+            "document[age]" => options[:age] || 0,
+            "_version" => Rack::Cachely::VERSION
           }
           rack[1].each do |key, value|
             data["document[headers][#{key}]"] = value

data/lib/rack-cachely/version.rb

@@ -1,5 +1,5 @@
 module Rack
   class Cachely
-    VERSION = "0.2.0"
+    VERSION = "0.2.1"
   end
 end

data/spec/rack-cachely/key_spec.rb

@@ -16,6 +16,16 @@ describe Rack::Cachely::Key do
       Rack::Cachely.config.ignore_query_params = ["A", "c"]
       key.to_s.should eql("http://example.com/foo/bar?b=B&d=D")
     end
+
+    context "no query string" do
+      
+      let(:request) { double(scheme: "http", host: "example.com", port: 80, script_name: "/foo", path_info: "/bar", query_string: "") }
+
+      it "does not have an empty ? at the end" do
+        key.to_s.should eql("http://example.com/foo/bar")
+      end
+
+    end
   
   end
 

data/spec/rack-cachely/store_spec.rb

@@ -10,7 +10,7 @@ describe Rack::Cachely::Store do
     it "retreives a page from the service" do
       response_mock = double(code: "200", body: "hello!")
       response_mock.stub!(:each_header).and_yield("Content-Type", "text/html")
-      Net::HTTP::Get.should_receive(:new).with("/v1/cache?url=http%3A%2F%2Fexample.com").and_return {
+      Net::HTTP::Get.should_receive(:new).with("/v1/cache?_version=#{Rack::Cachely::VERSION}&url=http%3A%2F%2Fexample.com").and_return {
         m = double
         m.should_receive(:basic_auth).with("1234567890", nil)
         m
@@ -47,7 +47,8 @@ describe Rack::Cachely::Store do
         "document[status]" => 200, 
         "document[body]" => "hello!", 
         "document[age]" => 30, 
-        "document[headers][Content-Type]" => "text/html"
+        "document[headers][Content-Type]" => "text/html",
+        "_version" => Rack::Cachely::VERSION
       }
     end
     

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-cachely
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-19 00:00:00.000000000 Z
+date: 2012-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack