suzanne.soy/travis-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use Rack::Request#user_agent

Browse Source
This commit is contained in:
Piotr Sarnacki 2012-11-13 02:32:00 +01:00
parent 7ad4ed1109
commit 973bd38f66
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/travis/web/app/mobile_redirect.rb
View File

@ -2,7 +2,7 @@ class Travis::Web::App::MobileRedirect < Struct.new(:app)
def call(env) def call(env)
request = Rack::Request.new env request = Rack::Request.new env
if request.params['mobile'] || env['HTTP_AGENT'] =~ /Mobile|webOS/ if request.params['mobile'] || request.user_agent.to_s =~ /Mobile|webOS/
location = 'https://secure.travis-ci.org' + request.fullpath location = 'https://secure.travis-ci.org' + request.fullpath
[301, { 'Content-Type' => 'text/plain', 'Location' => location }, []] [301, { 'Content-Type' => 'text/plain', 'Location' => location }, []]
else else

2
spec/mobile_redirect_spec.rb
View File

@ -6,7 +6,7 @@ describe Travis::Web::App::MobileRedirect do
let(:agent) { 'blah blah Mobile blablah' } let(:agent) { 'blah blah Mobile blablah' }
it 'redirects to secure.travis-ci.org' do it 'redirects to secure.travis-ci.org' do
get('/foo/bar?baz', {}, 'HTTP_AGENT' => agent).should be_redirect get('/foo/bar?baz', {}, 'HTTP_USER_AGENT' => agent).should be_redirect
last_response.headers['Location'].should == 'https://secure.travis-ci.org/foo/bar?baz' last_response.headers['Location'].should == 'https://secure.travis-ci.org/foo/bar?baz'
end end
end end