sprockets 2.1.0 → 2.1.2

data/README.md

@@ -354,9 +354,18 @@ submit a pull request.
 
 ## Version History ##
 
+**2.1.2** (November 20, 2011)
+
+* Disabled If-Modified-Since server checks. Fixes some browser caching issues when serving the asset body only. If-None-Match caching is sufficent.
+
+**2.1.1** (November 18, 2011)
+
+* Fix windows absolute path check bug.
+
 **2.1.0** (November 11, 2011)
 
-* Preserve directive whitespace.
+* Directive comment lines are now turned into empty lines instead of removed. This way line numbers in
+  CoffeeScript syntax errors are correct.
 * Performance and caching bug fixes.
 
 **2.0.3** (October 17, 2011)

data/lib/sprockets/base.rb

@@ -149,7 +149,7 @@ module Sprockets
       logical_path = path
       pathname     = Pathname.new(path)
 
-      if pathname.to_s =~ /^\//
+      if pathname.absolute?
         return unless stat(pathname)
         logical_path = attributes_for(pathname).logical_path
       else

data/lib/sprockets/cache/file_store.rb

@@ -6,14 +6,11 @@ module Sprockets
   module Cache
     # A simple file system cache store.
     #
-    #     environment.cache = Sprockets::Cache::FileStore.new("tmp/sprockets")
+    #     environment.cache = Sprockets::Cache::FileStore.new("/tmp")
     #
     class FileStore
       def initialize(root)
         @root = Pathname.new(root)
-
-        # Ensure directory exists
-        FileUtils.mkdir_p @root
       end
 
       # Lookup value in cache
@@ -24,6 +21,9 @@ module Sprockets
 
       # Save value to cache
       def []=(key, value)
+        # Ensure directory exists
+        FileUtils.mkdir_p @root.join(key).dirname
+
         @root.join(key).open('w') { |f| Marshal.dump(value, f)}
         value
       end

data/lib/sprockets/context.rb

@@ -78,7 +78,7 @@ module Sprockets
       pathname   = Pathname.new(path)
       attributes = environment.attributes_for(pathname)
 
-      if pathname.to_s =~ /^\//
+      if pathname.absolute?
         pathname
 
       elsif content_type = options[:content_type]

data/lib/sprockets/server.rb

@@ -53,9 +53,8 @@ module Sprockets
         # Return a 404 Not Found
         not_found_response
 
-      # Check request headers `HTTP_IF_MODIFIED_SINCE` and
-      # `HTTP_IF_NONE_MATCH` against the assets mtime and digest
-      elsif not_modified?(asset, env) || etag_match?(asset, env)
+      # Check request headers `HTTP_IF_NONE_MATCH` against the asset digest
+      elsif etag_match?(asset, env)
         logger.info "#{msg} 304 Not Modified (#{time_elapsed.call}ms)"
 
         # Return a 304 Not Modified
@@ -174,12 +173,6 @@ module Sprockets
           gsub('/',  '\\\\002f ')
       end
 
-      # Compare the requests `HTTP_IF_MODIFIED_SINCE` against the
-      # assets mtime
-      def not_modified?(asset, env)
-        env["HTTP_IF_MODIFIED_SINCE"] == asset.mtime.httpdate
-      end
-
       # Compare the requests `HTTP_IF_NONE_MATCH` against the assets digest
       def etag_match?(asset, env)
         env["HTTP_IF_NONE_MATCH"] == etag(asset)

data/lib/sprockets/version.rb

@@ -1,3 +1,3 @@
 module Sprockets
-  VERSION = "2.1.0"
+  VERSION = "2.1.2"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: sprockets
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 15
   prerelease: 
   segments: 
   - 2
   - 1
-  - 0
-  version: 2.1.0
+  - 2
+  version: 2.1.2
 platform: ruby
 authors: 
 - Sam Stephenson
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-11-11 00:00:00 -06:00
+date: 2011-11-20 00:00:00 -06:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency