prerender_rails 0.1.5 → 0.1.6
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/lib/prerender_rails.rb +2 -1
- data/prerender_rails.gemspec +1 -1
- data/test/lib/prerender_rails.rb +11 -3
- metadata +2 -2
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA1:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: b445e95eba89c9f2c17ef1bfba560d12d47ec6cb
|
|
4
|
+
data.tar.gz: 119ded420f8926a35ffc37bc61bac6872f226f5d
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 7416c6e2b014f91ef21acd2fbe5713052725262b5855c4d14ebeff86012c62702378ea6e3ad8545f8f2b3b3ffa7ccefefb1bd821af77ef54b019edaac4009e08
|
|
7
|
+
data.tar.gz: a4bdc217487a6814bf6d096d852733ada4a11a30f8a090925be4b618f343945f5b3a227272778af01d0ca3af7d4cb5377b0fd887f6984d398ebb35a8d68674f1
|
data/lib/prerender_rails.rb
CHANGED
|
@@ -66,7 +66,7 @@ module Rack
|
|
|
66
66
|
prerendered_response = get_prerendered_page_response(env)
|
|
67
67
|
|
|
68
68
|
if prerendered_response
|
|
69
|
-
response = Rack::Response.new(prerendered_response.body, prerendered_response.code,
|
|
69
|
+
response = Rack::Response.new(prerendered_response.body, prerendered_response.code, prerendered_response.header)
|
|
70
70
|
return response.finish
|
|
71
71
|
end
|
|
72
72
|
end
|
|
@@ -77,6 +77,7 @@ module Rack
|
|
|
77
77
|
def should_show_prerendered_page(env)
|
|
78
78
|
user_agent = env['HTTP_USER_AGENT']
|
|
79
79
|
return false if !user_agent
|
|
80
|
+
return false if env['REQUEST_METHOD'] != 'GET'
|
|
80
81
|
|
|
81
82
|
request = Rack::Request.new(env)
|
|
82
83
|
|
data/prerender_rails.gemspec
CHANGED
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
|
|
3
3
|
Gem::Specification.new do |spec|
|
|
4
4
|
spec.name = "prerender_rails"
|
|
5
|
-
spec.version = "0.1.
|
|
5
|
+
spec.version = "0.1.6"
|
|
6
6
|
spec.authors = ["Todd Hooper"]
|
|
7
7
|
spec.email = ["todd@collectiveip.com"]
|
|
8
8
|
spec.description = %q{Rails middleware to prerender your javascript heavy pages on the fly by a phantomjs service}
|
data/test/lib/prerender_rails.rb
CHANGED
|
@@ -13,13 +13,14 @@ describe Rack::Prerender do
|
|
|
13
13
|
@prerender = Rack::Prerender.new(@app)
|
|
14
14
|
end
|
|
15
15
|
|
|
16
|
-
it "should return a prerendered response for a crawler with the returned status code" do
|
|
16
|
+
it "should return a prerendered response for a crawler with the returned status code and headers" do
|
|
17
17
|
request = Rack::MockRequest.env_for "/", "HTTP_USER_AGENT" => bot
|
|
18
|
-
stub_request(:get, @prerender.build_api_url(request)).with(:headers => { 'User-Agent' => bot }).to_return(:body => "<html></html>", :status =>
|
|
18
|
+
stub_request(:get, @prerender.build_api_url(request)).with(:headers => { 'User-Agent' => bot }).to_return(:body => "<html></html>", :status => 301, :headers => { 'Location' => 'http://google.com'})
|
|
19
19
|
response = Rack::Prerender.new(@app).call(request)
|
|
20
20
|
|
|
21
21
|
assert_equal response[2].body, ["<html></html>"]
|
|
22
|
-
assert_equal response[2].status,
|
|
22
|
+
assert_equal response[2].status, 301
|
|
23
|
+
assert_equal response[2].headers, { 'location' => 'http://google.com', 'Content-Length' => '13'}
|
|
23
24
|
end
|
|
24
25
|
|
|
25
26
|
it "should return a prerendered reponse if user is a bot by checking for _escaped_fragment_" do
|
|
@@ -37,6 +38,13 @@ describe Rack::Prerender do
|
|
|
37
38
|
assert_equal response[2], ""
|
|
38
39
|
end
|
|
39
40
|
|
|
41
|
+
it "should continue to app routes if the request is not a GET" do
|
|
42
|
+
request = Rack::MockRequest.env_for "/path?_escaped_fragment_=", { "HTTP_USER_AGENT" => user, "REQUEST_METHOD" => "POST" }
|
|
43
|
+
response = Rack::Prerender.new(@app).call(request)
|
|
44
|
+
|
|
45
|
+
assert_equal response[2], ""
|
|
46
|
+
end
|
|
47
|
+
|
|
40
48
|
it "should continue to app routes if user is not a bot by checking agent string" do
|
|
41
49
|
request = Rack::MockRequest.env_for "/", "HTTP_USER_AGENT" => user
|
|
42
50
|
response = Rack::Prerender.new(@app).call(request)
|
metadata
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: prerender_rails
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.1.
|
|
4
|
+
version: 0.1.6
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- Todd Hooper
|
|
8
8
|
autorequire:
|
|
9
9
|
bindir: bin
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date: 2013-
|
|
11
|
+
date: 2013-11-08 00:00:00.000000000 Z
|
|
12
12
|
dependencies:
|
|
13
13
|
- !ruby/object:Gem::Dependency
|
|
14
14
|
name: rack
|