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

Merge pull request #27 from neo/master

Browse Source 
Handle Rails apps running on relative root
This commit is contained in:
John Bintz 2013-04-02 08:22:50 -07:00
commit 03ea0608cd
3 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/rack/livereload.rb
View File

@ -77,8 +77,10 @@ module Rack
if !headers['X-Rack-LiveReload'] && line['<head'] if !headers['X-Rack-LiveReload'] && line['<head']
host_to_use = (@options[:host] || env['HTTP_HOST'] || 'localhost').gsub(%r{:.*}, '') host_to_use = (@options[:host] || env['HTTP_HOST'] || 'localhost').gsub(%r{:.*}, '')
app_root = ENV['RAILS_RELATIVE_URL_ROOT'] || ''
if use_vendored? if use_vendored?
src = LIVERELOAD_JS_PATH.dup + "?host=#{host_to_use}" src = "#{app_root}#{LIVERELOAD_JS_PATH.dup}?host=#{host_to_use}"
else else
src = livereload_local_uri.dup.gsub('localhost', host_to_use) + '?' src = livereload_local_uri.dup.gsub('localhost', host_to_use) + '?'
end end

4
skel/livereload.html.erb
View File

@ -5,8 +5,8 @@
WEB_SOCKET_FORCE_FLASH = true; WEB_SOCKET_FORCE_FLASH = true;
<% end %> <% end %>
</script> </script>
<script type="text/javascript" src="/__rack/swfobject.js"></script> <script type="text/javascript" src="<%= app_root %>/__rack/swfobject.js"></script>
<script type="text/javascript" src="/__rack/web_socket.js"></script> <script type="text/javascript" src="<%= app_root %>/__rack/web_socket.js"></script>
<% end %> <% end %>
<script type="text/javascript"> <script type="text/javascript">
RACK_LIVERELOAD_PORT = <%= @port %>; RACK_LIVERELOAD_PORT = <%= @port %>;

9
spec/rack/livereload_spec.rb
View File

@ -161,6 +161,15 @@ describe Rack::LiveReload do
body_dom.at_css("script:last-child")[:insert].should == "before" body_dom.at_css("script:last-child")[:insert].should == "before"
end end
context 'when a relative URL root is specified' do
before do
ENV['RAILS_RELATIVE_URL_ROOT'] = '/a_relative_path'
end
it 'should prepend the relative path to the script src' do
body_dom.at_css("script:eq(5)")[:src].should match(%r{^/a_relative_path/})
end
end
end end
describe "LIVERELOAD_PORT value" do describe "LIVERELOAD_PORT value" do