Add parens for 'Ambiguous first argument' warnings in jruby

sinatra · Sep 29, 2011 · 37b7b73 · 37b7b73
1 parent fff0570
commit 37b7b73
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/rack/protection/ip_spoofing.rb b/lib/rack/protection/ip_spoofing.rb
@@ -13,7 +13,7 @@ class IPSpoofing < Base
 
       def accepts?(env)
         return true unless env.include? 'HTTP_X_FORWARDED_FOR'
-        ips = env['HTTP_X_FORWARDED_FOR'].split /\s*,\s*/
+        ips = env['HTTP_X_FORWARDED_FOR'].split(/\s*,\s*/)
         return false if env.include? 'HTTP_CLIENT_IP' and not ips.include? env['HTTP_CLIENT_IP']
         return false if env.include? 'HTTP_X_REAL_IP' and not ips.include? env['HTTP_X_REAL_IP']
         true

diff --git a/lib/rack/protection/path_traversal.rb b/lib/rack/protection/path_traversal.rb
@@ -22,7 +22,7 @@ def cleanup(path)
         return cleanup("/" << path)[1..-1] unless path[0] == ?/
         escaped = ::File.expand_path path.gsub('%2e', '.').gsub('%2f', '/')
         escaped << '/' if escaped[-1] != ?/ and path =~ /\/\.{0,2}$/
-        escaped.gsub /\/\/+/, '/'
+        escaped.gsub(/\/\/+/, '/')
       end
     end
   end