prerender_rails 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3ee24bb70559470ecc22d9d53d4f25cea8d4919e
-  data.tar.gz: 23ba68c813c0fb612dce060dc80c1f63de794192
+  metadata.gz: 0dab2fb1c0fb1a232b5901dde71f986ca4000c65
+  data.tar.gz: 675b2dfbdede57ad5f448377aa964e304f5fea30
 SHA512:
-  metadata.gz: e67ad893ffd7b2a1976076425fb9cb6750c893e2b316a9016438b74d262231d7f48ff1ad6e272ba646cfb03cedc39a59702d36073e30b0808e12a65a9292a712
-  data.tar.gz: b71f81d28e7e98e43101783e408d38851a26c3fd0831a6d80e8ad81373207b03dfe28342468cb06142b16061b7b61e224902066d514ed75d76e76c3a9d47aca5
+  metadata.gz: 748be26bea44cae70d7c0f5197f1e54aeede93c8951868e70d5acc7d6a620fb86deac649a8a71b4bd07c81e26ab73af9b79c09316e5362ec8a0d60afaa35bfc2
+  data.tar.gz: e34eb14973e15ec46bbbf9214aaad0bc688a8d9edc58b4148ec8828d2d447d55700bd7b0ebf7f8c8af56138340d1510a8d070bc7eba6379dd3dbc3b6e4293b5c

data/lib/prerender_rails.rb

@@ -121,10 +121,10 @@ module Rack
       is_requesting_prerendered_page = true if buffer_agent
 
       #if it is a bot and is requesting a resource...dont prerender
-      return false if @extensions_to_ignore.any? { |extension| request.path.include? extension }
+      return false if @extensions_to_ignore.any? { |extension| request.fullpath.include? extension }
 
       #if it is a bot and not requesting a resource and is not whitelisted...dont prerender
-      return false if @options[:whitelist].is_a?(Array) && @options[:whitelist].all? { |whitelisted| !Regexp.new(whitelisted).match(request.path) }
+      return false if @options[:whitelist].is_a?(Array) && @options[:whitelist].all? { |whitelisted| !Regexp.new(whitelisted).match(request.fullpath) }
 
       #if it is a bot and not requesting a resource and is not blacklisted(url or referer)...dont prerender
       if @options[:blacklist].is_a?(Array) && @options[:blacklist].any? { |blacklisted|
@@ -132,7 +132,7 @@ module Rack
           blacklistedReferer = false
           regex = Regexp.new(blacklisted)
 
-          blacklistedUrl = !!regex.match(request.path)
+          blacklistedUrl = !!regex.match(request.fullpath)
           blacklistedReferer = !!regex.match(request.referer) if request.referer
 
           blacklistedUrl || blacklistedReferer
@@ -212,7 +212,7 @@ module Rack
       cached_render = @options[:before_render].call(env)
 
       if cached_render && cached_render.is_a?(String)
-        Rack::Response.new(cached_render, 200, [])
+        Rack::Response.new(cached_render, 200, { 'Content-Type' => 'text/html; charset=utf-8' })
       elsif cached_render && cached_render.is_a?(Rack::Response)
         cached_render
       else

data/prerender_rails.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "prerender_rails"
-  spec.version       = "1.2.0"
+  spec.version       = "1.3.0"
   spec.authors       = ["Todd Hooper"]
   spec.email         = ["todd@prerender.io"]
   spec.description   = %q{Rails middleware to prerender your javascript heavy pages on the fly by a phantomjs service}

data/test/lib/prerender_rails.rb

@@ -90,6 +90,12 @@ describe Rack::Prerender do
     assert_equal "", response[2]
   end
 
+  it "should continue to app routes if the hashbang url is part of the regex specific blacklist" do
+    request = Rack::MockRequest.env_for "?_escaped_fragment_=/search/things/123/page", "HTTP_USER_AGENT" => bot
+    response = Rack::Prerender.new(@app, blacklist: ['/search', '/help']).call(request)
+
+    assert_equal "", response[2]
+  end
 
   it "should return a prerendered response if the url is not part of the regex specific blacklist" do
     request = Rack::MockRequest.env_for "/profile/search/blah", "HTTP_USER_AGENT" => bot

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prerender_rails
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Todd Hooper
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-17 00:00:00.000000000 Z
+date: 2016-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack