rack-cache 0.2.0 → 0.3.0

data/rack-cache.gemspec

@@ -3,8 +3,8 @@ Gem::Specification.new do |s|
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
 
   s.name = 'rack-cache'
-  s.version = '0.2.0'
-  s.date = '2008-10-24'
+  s.version = '0.3.0'
+  s.date = '2008-12-28'
 
   s.description = "HTTP Caching for Rack"
   s.summary     = "HTTP Caching for Rack"
@@ -26,6 +26,7 @@ Gem::Specification.new do |s|
     doc/layout.html.erb
     doc/license.markdown
     doc/rack-cache.css
+    doc/server.ru
     doc/storage.markdown
     lib/rack/cache.rb
     lib/rack/cache/config.rb
@@ -62,11 +63,11 @@ Gem::Specification.new do |s|
 
   s.test_files = s.files.select {|path| path =~ /^test\/.*_test.rb/}
 
-  s.extra_rdoc_files = %w[COPYING]
+  s.extra_rdoc_files = %w[README COPYING TODO CHANGES]
   s.add_dependency 'rack', '~> 0.4'
 
   s.has_rdoc = true
-  s.homepage = "http://github.com/rtomayko/rack-cache"
+  s.homepage = "http://tomayko.com/src/rack-cache/"
   s.rdoc_options = ["--line-numbers", "--inline-source", "--title", "Rack::Cache", "--main", "Rack::Cache"]
   s.require_paths = %w[lib]
   s.rubyforge_project = 'wink'

data/test/cache_test.rb

@@ -18,8 +18,8 @@ describe 'Rack::Cache::new' do
   end
   it 'sets options provided in the options Hash' do
     object = Rack::Cache.new(@app, :foo => 'bar', 'foo.bar' => 'bling')
-    object.options['foo.bar'].should.be == 'bling'
-    object.options['rack-cache.foo'].should.be == 'bar'
+    object.options['foo.bar'].should.equal 'bling'
+    object.options['rack-cache.foo'].should.equal 'bar'
   end
   it 'takes a block; executes it during initialization' do
     state, block_scope = 'not invoked', nil
@@ -29,7 +29,7 @@ describe 'Rack::Cache::new' do
         state = 'invoked'
         should.respond_to :on
       end
-    state.should.be == 'invoked'
+    state.should.equal 'invoked'
     object.should.be block_scope
   end
 end

data/test/context_test.rb

@@ -15,24 +15,98 @@ describe 'Rack::Cache::Context' do
     response.headers.should.not.include 'Age'
   end
 
-  it 'passes on requests with Authorization' do
-    respond_with 200
+  it 'does not cache with Authorization request header and non public response' do
+    respond_with 200, 'Etag' => '"FOO"'
     get '/', 'HTTP_AUTHORIZATION' => 'basic foobarbaz'
 
     app.should.be.called
     response.should.be.ok
-    cache.should.a.performed :pass
+    response.headers['Cache-Control'].should.equal 'private'
+    cache.should.a.performed :miss
+    cache.should.a.performed :fetch
+    cache.should.a.not.performed :store
+    cache.should.a.performed :deliver
     response.headers.should.not.include 'Age'
   end
 
-  it 'passes on requests with a Cookie' do
+  it 'does cache with Authorization request header and public response' do
+    respond_with 200, 'Cache-Control' => 'public', 'ETag' => '"FOO"'
+    get '/', 'HTTP_AUTHORIZATION' => 'basic foobarbaz'
+
+    app.should.be.called
+    response.should.be.ok
+    cache.should.a.performed :miss
+    cache.should.a.performed :fetch
+    cache.should.a.performed :store
+    response.headers.should.include 'Age'
+    response.headers['Cache-Control'].should.equal 'public'
+  end
+
+  it 'does not cache with Cookie header and non public response' do
+    respond_with 200, 'Etag' => '"FOO"'
+    get '/', 'HTTP_COOKIE' => 'foo=bar'
+
+    app.should.be.called
+    response.should.be.ok
+    response.headers['Cache-Control'].should.equal 'private'
+    cache.should.a.performed :miss
+    cache.should.a.performed :fetch
+    cache.should.a.not.performed :store
+    cache.should.a.performed :deliver
+    response.headers.should.not.include 'Age'
+  end
+
+  it 'does not cache requests with a Cookie header' do
     respond_with 200
     get '/', 'HTTP_COOKIE' => 'foo=bar'
 
     response.should.be.ok
     app.should.be.called
-    cache.should.a.performed :pass
+    cache.should.a.performed :miss
+    cache.should.a.performed :fetch
+    cache.should.a.performed :deliver
     response.headers.should.not.include 'Age'
+    response.headers['Cache-Control'].should.equal 'private'
+  end
+
+  it 'responds with 304 when If-Modified-Since matches Last-Modified' do
+    timestamp = Time.now.httpdate
+    respond_with do |req,res|
+      res.status = 200
+      res['Last-Modified'] = timestamp
+      res['Content-Type'] = 'text/plain'
+      res.body = ['Hello World']
+    end
+
+    get '/',
+      'HTTP_IF_MODIFIED_SINCE' => timestamp
+    app.should.be.called
+    response.status.should.equal 304
+    response.headers.should.not.include 'Content-Length'
+    response.headers.should.not.include 'Content-Type'
+    response.body.should.empty
+    cache.should.a.performed :miss
+    cache.should.a.performed :store
+  end
+
+  it 'responds with 304 when If-None-Match matches ETag' do
+    respond_with do |req,res|
+      res.status = 200
+      res['Etag'] = '12345'
+      res['Content-Type'] = 'text/plain'
+      res.body = ['Hello World']
+    end
+
+    get '/',
+      'HTTP_IF_NONE_MATCH' => '12345'
+    app.should.be.called
+    response.status.should.equal 304
+    response.headers.should.not.include 'Content-Length'
+    response.headers.should.not.include 'Content-Type'
+    response.headers.should.include 'Etag'
+    response.body.should.empty
+    cache.should.a.performed :miss
+    cache.should.a.performed :store
   end
 
   it 'caches requests when Cache-Control request header set to no-cache' do
@@ -41,7 +115,7 @@ describe 'Rack::Cache::Context' do
 
     response.should.be.ok
     cache.should.a.performed :store
-    response.headers.should.not.include 'Age'
+    response.headers.should.include 'Age'
   end
 
   it 'fetches response from backend when cache misses' do
@@ -51,7 +125,7 @@ describe 'Rack::Cache::Context' do
     response.should.be.ok
     cache.should.a.performed :miss
     cache.should.a.performed :fetch
-    response.headers.should.not.include 'Age'
+    response.headers.should.include 'Age'
   end
 
   [(201..202),(204..206),(303..305),(400..403),(405..409),(411..417),(500..505)].each do |range|
@@ -61,7 +135,7 @@ describe 'Rack::Cache::Context' do
         get '/'
 
         cache.should.a.not.performed :store
-        response.status.should.be == response_code
+        response.status.should.equal response_code
         response.headers.should.not.include 'Age'
       end
     end
@@ -94,7 +168,7 @@ describe 'Rack::Cache::Context' do
 
     response.should.be.ok
     cache.should.a.performed :store
-    response.headers.should.not.include 'Age'
+    response.headers.should.include 'Age'
   end
 
   it 'caches responses with an Expiration header' do
@@ -102,13 +176,13 @@ describe 'Rack::Cache::Context' do
     get '/'
 
     response.should.be.ok
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
     response.headers.should.include 'Date'
-    response['Age'].should.be.nil
-    response['X-Content-Digest'].should.be.nil
+    response['Age'].should.not.be.nil
+    response['X-Content-Digest'].should.not.be.nil
     cache.should.a.performed :miss
     cache.should.a.performed :store
-    cache.metastore.to_hash.keys.length.should.be == 1
+    cache.metastore.to_hash.keys.length.should.equal 1
   end
 
   it 'caches responses with a max-age directive' do
@@ -116,13 +190,27 @@ describe 'Rack::Cache::Context' do
     get '/'
 
     response.should.be.ok
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
     response.headers.should.include 'Date'
-    response['Age'].should.be.nil
-    response['X-Content-Digest'].should.be.nil
+    response['Age'].should.not.be.nil
+    response['X-Content-Digest'].should.not.be.nil
     cache.should.a.performed :miss
     cache.should.a.performed :store
-    cache.metastore.to_hash.keys.length.should.be == 1
+    cache.metastore.to_hash.keys.length.should.equal 1
+  end
+
+  it 'caches responses with a s-maxage directive' do
+    respond_with 200, 'Cache-Control' => 's-maxage=5'
+    get '/'
+
+    response.should.be.ok
+    response.body.should.equal 'Hello World'
+    response.headers.should.include 'Date'
+    response['Age'].should.not.be.nil
+    response['X-Content-Digest'].should.not.be.nil
+    cache.should.a.performed :miss
+    cache.should.a.performed :store
+    cache.metastore.to_hash.keys.length.should.equal 1
   end
 
   it 'caches responses with a Last-Modified validator but no freshness information' do
@@ -130,7 +218,7 @@ describe 'Rack::Cache::Context' do
     get '/'
 
     response.should.be.ok
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
     cache.should.a.performed :miss
     cache.should.a.performed :store
   end
@@ -140,7 +228,7 @@ describe 'Rack::Cache::Context' do
     get '/'
 
     response.should.be.ok
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
     cache.should.a.performed :miss
     cache.should.a.performed :store
   end
@@ -156,17 +244,17 @@ describe 'Rack::Cache::Context' do
     response.headers.should.include 'Date'
     cache.should.a.performed :miss
     cache.should.a.performed :store
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
 
     get '/'
     response.should.be.ok
     app.should.not.be.called
-    response['Date'].should.be == responses.first['Date']
-    response['Age'].to_i.should.be > 0
+    response['Date'].should.equal responses.first['Date']
+    response['Age'].to_i.should.satisfy { |age| age > 0 }
     response['X-Content-Digest'].should.not.be.nil
     cache.should.a.performed :hit
     cache.should.a.not.performed :fetch
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
   end
 
   it 'hits cached response with max-age directive' do
@@ -180,17 +268,73 @@ describe 'Rack::Cache::Context' do
     response.headers.should.include 'Date'
     cache.should.a.performed :miss
     cache.should.a.performed :store
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
 
     get '/'
     response.should.be.ok
     app.should.not.be.called
-    response['Date'].should.be == responses.first['Date']
-    response['Age'].to_i.should.be > 0
+    response['Date'].should.equal responses.first['Date']
+    response['Age'].to_i.should.satisfy { |age| age > 0 }
     response['X-Content-Digest'].should.not.be.nil
     cache.should.a.performed :hit
     cache.should.a.not.performed :fetch
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
+  end
+
+  it 'hits cached response with s-maxage directive' do
+    respond_with 200,
+      'Date' => (Time.now - 5).httpdate,
+      'Cache-Control' => 's-maxage=10, max-age=0'
+
+    get '/'
+    app.should.be.called
+    response.should.be.ok
+    response.headers.should.include 'Date'
+    cache.should.a.performed :miss
+    cache.should.a.performed :store
+    response.body.should.equal 'Hello World'
+
+    get '/'
+    response.should.be.ok
+    app.should.not.be.called
+    response['Date'].should.equal responses.first['Date']
+    response['Age'].to_i.should.satisfy { |age| age > 0 }
+    response['X-Content-Digest'].should.not.be.nil
+    cache.should.a.performed :hit
+    cache.should.a.not.performed :fetch
+    response.body.should.equal 'Hello World'
+  end
+
+  it 'assigns default_ttl when response has no freshness information' do
+    respond_with 200
+
+    get '/', 'rack-cache.default_ttl' => 10
+    app.should.be.called
+    response.should.be.ok
+    cache.should.a.performed :miss
+    cache.should.a.performed :store
+    response.body.should.equal 'Hello World'
+    response['Cache-Control'].should.include 's-maxage=10'
+
+    get '/', 'rack-cache.default_ttl' => 10
+    response.should.be.ok
+    app.should.not.be.called
+    cache.should.a.performed :hit
+    cache.should.a.not.performed :fetch
+    response.body.should.equal 'Hello World'
+  end
+
+  it 'does not assign default_ttl when response has must-revalidate directive' do
+    respond_with 200,
+      'Cache-Control' => 'must-revalidate'
+
+    get '/', 'rack-cache.default_ttl' => 10
+    app.should.be.called
+    response.should.be.ok
+    cache.should.a.performed :miss
+    cache.should.a.not.performed :store
+    response['Cache-Control'].should.not.include 's-maxage'
+    response.body.should.equal 'Hello World'
   end
 
   it 'fetches full response when cache stale and no validators present' do
@@ -201,27 +345,27 @@ describe 'Rack::Cache::Context' do
     app.should.be.called
     response.should.be.ok
     response.headers.should.include 'Date'
-    response.headers.should.not.include 'X-Content-Digest'
-    response['Age'].should.be.nil
+    response.headers.should.include 'X-Content-Digest'
+    response.headers.should.include 'Age'
     cache.should.a.performed :miss
     cache.should.a.performed :store
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
 
     # go in and play around with the cached metadata directly ...
-    cache.metastore.to_hash.values.length.should.be == 1
+    cache.metastore.to_hash.values.length.should.equal 1
     cache.metastore.to_hash.values.first.first[1]['Expires'] = Time.now.httpdate
 
     # build subsequent request; should be found but miss due to freshness
     get '/'
     app.should.be.called
     response.should.be.ok
-    response['Age'].to_i.should.be == 0
-    response['X-Content-Digest'].should.be.nil
+    response['Age'].to_i.should.equal 0
+    response.headers.should.include 'X-Content-Digest'
     cache.should.a.not.performed :hit
     cache.should.a.not.performed :miss
     cache.should.a.performed :fetch
     cache.should.a.performed :store
-    response.body.should.be == 'Hello World'
+    response.body.should.equal 'Hello World'
   end
 
   it 'validates cached responses with Last-Modified and no freshness information' do
@@ -239,8 +383,8 @@ describe 'Rack::Cache::Context' do
     app.should.be.called
     response.should.be.ok
     response.headers.should.include 'Last-Modified'
-    response.headers.should.not.include 'X-Content-Digest'
-    response.body.should.be == 'Hello World'
+    response.headers.should.include 'X-Content-Digest'
+    response.body.should.equal 'Hello World'
     cache.should.a.performed :miss
     cache.should.a.performed :store
 
@@ -250,9 +394,9 @@ describe 'Rack::Cache::Context' do
     response.should.be.ok
     response.headers.should.include 'Last-Modified'
     response.headers.should.include 'X-Content-Digest'
-    response['Age'].to_i.should.be == 0
-    response['X-Origin-Status'].should.be == '304'
-    response.body.should.be == 'Hello World'
+    response['Age'].to_i.should.equal 0
+    response['X-Origin-Status'].should.equal '304'
+    response.body.should.equal 'Hello World'
     cache.should.a.not.performed :miss
     cache.should.a.performed :fetch
     cache.should.a.performed :store
@@ -273,8 +417,8 @@ describe 'Rack::Cache::Context' do
     app.should.be.called
     response.should.be.ok
     response.headers.should.include 'Etag'
-    response.headers.should.not.include 'X-Content-Digest'
-    response.body.should.be == 'Hello World'
+    response.headers.should.include 'X-Content-Digest'
+    response.body.should.equal 'Hello World'
     cache.should.a.performed :miss
     cache.should.a.performed :store
 
@@ -284,9 +428,9 @@ describe 'Rack::Cache::Context' do
     response.should.be.ok
     response.headers.should.include 'Etag'
     response.headers.should.include 'X-Content-Digest'
-    response['Age'].to_i.should.be == 0
-    response['X-Origin-Status'].should.be == '304'
-    response.body.should.be == 'Hello World'
+    response['Age'].to_i.should.equal 0
+    response['X-Origin-Status'].should.equal '304'
+    response.body.should.equal 'Hello World'
     cache.should.a.not.performed :miss
     cache.should.a.performed :fetch
     cache.should.a.performed :store
@@ -308,22 +452,58 @@ describe 'Rack::Cache::Context' do
 
     # first request should fetch from backend and store in cache
     get '/'
-    response.status.should.be == 200
-    response.body.should.be == 'first response'
+    response.status.should.equal 200
+    response.body.should.equal 'first response'
 
     # second request is validated, is invalid, and replaces cached entry
     get '/'
-    response.status.should.be == 200
-    response.body.should.be == 'second response'
+    response.status.should.equal 200
+    response.body.should.equal 'second response'
 
     # third respone is validated, valid, and returns cached entry
     get '/'
-    response.status.should.be == 200
-    response.body.should.be == 'second response'
+    response.status.should.equal 200
+    response.body.should.equal 'second response'
+
+    count.should.equal 3
+  end
+
+  it 'passes HEAD requests through directly on pass' do
+    respond_with do |req,res|
+      res.status = 200
+      res.body = []
+      req.request_method.should.equal 'HEAD'
+    end
+
+    cache_config do
+      on(:receive) { pass! }
+    end
+
+    head '/'
+    app.should.be.called
+    response.body.should.equal ''
+  end
+
+  it 'uses cache to respond to HEAD requests when fresh' do
+    respond_with do |req,res|
+      res['Cache-Control'] = 'max-age=10'
+      res.body = ['Hello World']
+      req.request_method.should.not.equal 'HEAD'
+    end
+
+    get '/'
+    app.should.be.called
+    response.status.should.equal 200
+    response.body.should.equal 'Hello World'
 
-    count.should.be == 3
+    head '/'
+    app.should.not.be.called
+    response.status.should.equal 200
+    response.body.should.equal ''
+    response['Content-Length'].should.equal 'Hello World'.length.to_s
   end
 
+
   describe 'with responses that include a Vary header' do
     before(:each) do
       count = 0
@@ -340,7 +520,7 @@ describe 'Rack::Cache::Context' do
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/1.0'
       response.should.be.ok
-      response.body.should.be == 'Bob/1.0'
+      response.body.should.equal 'Bob/1.0'
       cache.should.a.performed :miss
       cache.should.a.performed :store
 
@@ -348,7 +528,7 @@ describe 'Rack::Cache::Context' do
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/1.0'
       response.should.be.ok
-      response.body.should.be == 'Bob/1.0'
+      response.body.should.equal 'Bob/1.0'
       cache.should.a.performed :hit
       cache.should.a.not.performed :fetch
       response.headers.should.include 'X-Content-Digest'
@@ -359,36 +539,36 @@ describe 'Rack::Cache::Context' do
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/1.0'
       response.should.be.ok
-      response.body.should.be == 'Bob/1.0'
-      response['X-Response-Count'].should.be == '1'
+      response.body.should.equal 'Bob/1.0'
+      response['X-Response-Count'].should.equal '1'
 
       get '/',
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/2.0'
       cache.should.a.performed :miss
       cache.should.a.performed :store
-      response.body.should.be == 'Bob/2.0'
-      response['X-Response-Count'].should.be == '2'
+      response.body.should.equal 'Bob/2.0'
+      response['X-Response-Count'].should.equal '2'
 
       get '/',
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/1.0'
       cache.should.a.performed :hit
-      response.body.should.be == 'Bob/1.0'
-      response['X-Response-Count'].should.be == '1'
+      response.body.should.equal 'Bob/1.0'
+      response['X-Response-Count'].should.equal '1'
 
       get '/',
         'HTTP_ACCEPT' => 'text/html',
         'HTTP_USER_AGENT' => 'Bob/2.0'
       cache.should.a.performed :hit
-      response.body.should.be == 'Bob/2.0'
-      response['X-Response-Count'].should.be == '2'
+      response.body.should.equal 'Bob/2.0'
+      response['X-Response-Count'].should.equal '2'
 
       get '/',
         'HTTP_USER_AGENT' => 'Bob/2.0'
       cache.should.a.performed :miss
-      response.body.should.be == 'Bob/2.0'
-      response['X-Response-Count'].should.be == '3'
+      response.body.should.equal 'Bob/2.0'
+      response['X-Response-Count'].should.equal '3'
     end
   end
 
@@ -401,7 +581,7 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! }
       end
       get '/'
-      response.status.should.be == 500
+      response.status.should.equal 500
       response.body.should.be.empty
       cache.should.a.performed :error
     end
@@ -411,7 +591,7 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! 505 }
       end
       get '/'
-      response.status.should.be == 505
+      response.status.should.equal 505
     end
 
     it 'sets the status and headers with args: status, Hash' do
@@ -419,8 +599,8 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! 504, 'Content-Type' => 'application/x-foo' }
       end
       get '/'
-      response.status.should.be == 504
-      response['Content-Type'].should.be == 'application/x-foo'
+      response.status.should.equal 504
+      response['Content-Type'].should.equal 'application/x-foo'
       response.body.should.be.empty
     end
 
@@ -429,8 +609,8 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! 503, 'foo bar baz' }
       end
       get '/'
-      response.status.should.be == 503
-      response.body.should.be == 'foo bar baz'
+      response.status.should.equal 503
+      response.body.should.equal 'foo bar baz'
     end
 
     it 'sets the status and body with args: status, Array' do
@@ -438,8 +618,8 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! 503, ['foo bar baz'] }
       end
       get '/'
-      response.status.should.be == 503
-      response.body.should.be == 'foo bar baz'
+      response.status.should.equal 503
+      response.body.should.equal 'foo bar baz'
     end
 
     it 'fires the error event before finishing' do
@@ -448,16 +628,16 @@ describe 'Rack::Cache::Context' do
         on(:receive) { error! }
         on(:error) {
           fired = true
-          response.status.should.be == 500
+          response.status.should.equal 500
           response['Content-Type'] = 'application/x-foo'
           response.body = ['overridden response body']
         }
       end
       get '/'
       fired.should.be true
-      response.status.should.be == 500
-      response.body.should.be == 'overridden response body'
-      response['Content-Type'].should.be == 'application/x-foo'
+      response.status.should.equal 500
+      response.body.should.equal 'overridden response body'
+      response['Content-Type'].should.equal 'application/x-foo'
     end
 
   end