Addressable::URI.parse might return nil

2012-12-06 15:34:13 +01:00 · 2012-12-06 15:34:13 +01:00 · afad81f927
commit afad81f927
parent ee1f9d899d
1 changed files with 1 additions and 1 deletions
--- a/lib/travis/api/app/endpoint/authorization.rb
+++ b/lib/travis/api/app/endpoint/authorization.rb
@ -253,7 +253,7 @@ class Travis::Api::App
        end

        def target_ok?(target_origin)
-          uri = Addressable::URI.parse(target_origin)
+          return unless uri = Addressable::URI.parse(target_origin)
          if uri.host =~ /\A(.+\.)?travis-ci\.(com|org)\Z/
            uri.scheme == 'https'
          elsif uri == 'localhost' or uri == '127.0.0.1'