github-migration/rack-livereload
1
0
Fork 0
Code Issues 11 Pull Requests 4 Packages Projects Releases Wiki Activity

merge

Browse Source
This commit is contained in:
John Bintz 2012-03-02 09:27:12 -05:00
commit 7733963658
4 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.gitignore vendored
View File

@ -2,3 +2,4 @@
.bundle
Gemfile.lock
pkg/*
*.orig

2
lib/rack-livereload.rb
View File

@ -1,6 +1,6 @@
require "rack/livereload"
class Rack::LiveReload
VERSION = '0.3.3'
VERSION = '0.3.4'
end

4
lib/rack/livereload.rb
View File

@ -52,7 +52,7 @@ module Rack
new_body = body.dup
if !ignored?(env['PATH_INFO']) && !bad_browser?(env['HTTP_USER_AGENT'])
if headers['Content-Type'][%r{text/html}]
if headers['Content-Type'] && headers['Content-Type'][%r{text/html}]
content_length = 0
new_body = []
@ -78,7 +78,7 @@ module Rack
headers["X-Rack-LiveReload"] = '1'
end
content_length += line.length
content_length += line.bytesize
new_body << line
end

12
spec/rack/livereload_spec.rb
View File

@ -53,6 +53,18 @@ describe Rack::LiveReload do
end
end
context 'unknown Content-Type' do
let(:ret) { [ 200, {}, [ 'hey ho' ] ] }
before do
app.stubs(:call).with(env).returns(ret)
end
it 'should not break' do
middleware.call(env).should_not raise_error(NoMethodError, /You have a nil object/)
end
end
context 'text/html' do
before do
app.stubs(:call).with(env).returns([ 200, { 'Content-Type' => 'text/html', 'Content-Length' => 0 }, [ '<head></head>' ] ])