Fix warnings raised by URI 1.0.1

Related to:
https://www.github.com/ruby/uri/issues/125
#23260

Author: jrafanie

app/models/external_url.rb

@@ -2,5 +2,5 @@ class ExternalUrl < ApplicationRecord
   belongs_to :resource, :polymorphic => true
   belongs_to :user
 
-  validates :url, :format => URI::DEFAULT_PARSER.make_regexp, :allow_nil => false
+  validates :url, :format => URI::RFC2396_PARSER.make_regexp, :allow_nil => false
 end

app/models/file_depot_ftp.rb

@@ -127,7 +127,7 @@ def destination_path
 
   def base_path
     # uri: "ftp://ftp.example.com/incoming" => #<Pathname:incoming>
-    path = URI(URI::DEFAULT_PARSER.escape(uri)).path
+    path = URI(URI::RFC2396_PARSER.escape(uri)).path
     Pathname.new(path)
   end
 

app/models/git_repository.rb

@@ -9,7 +9,7 @@ class GitRepository < ApplicationRecord
 
   attr_reader :git_lock
 
-  validates :url, :format => Regexp.union(URI::DEFAULT_PARSER.make_regexp(%w[http https file ssh]), /\A[-\w:.]+@.*:/), :allow_nil => false
+  validates :url, :format => Regexp.union(URI::RFC2396_PARSER.make_regexp(%w[http https file ssh]), /\A[-\w:.]+@.*:/), :allow_nil => false
 
   default_value_for :verify_ssl, OpenSSL::SSL::VERIFY_PEER
   validates :verify_ssl, :inclusion => {:in => [OpenSSL::SSL::VERIFY_NONE, OpenSSL::SSL::VERIFY_PEER]}

app/models/log_file.rb

@@ -27,7 +27,7 @@ def relative_path_for_upload(loc_file)
   # Base is the URI defined by the user
   # loc_file is the name of the original file
   def build_log_uri(base_uri, loc_file)
-    scheme, userinfo, host, port, registry, path, opaque, query, fragment = URI.split(URI::DEFAULT_PARSER.escape(base_uri))
+    scheme, userinfo, host, port, registry, path, opaque, query, fragment = URI.split(URI::RFC2396_PARSER.escape(base_uri))
 
     # Convert encoded spaces back to spaces
     path.gsub!('%20', ' ')
@@ -211,7 +211,7 @@ def get_post_method(uri)
     # Strip any leading and trailing whitespace
     uri.strip!
 
-    URI.split(URI::DEFAULT_PARSER.escape(uri))[0]
+    URI.split(URI::RFC2396_PARSER.escape(uri))[0]
   end
 
   def legacy_depot_hash

app/models/mixins/file_depot_mixin.rb

@@ -57,7 +57,7 @@ def get_uri_prefix(uri_str)
       # Strip any leading and trailing whitespace
       uri_str.strip!
 
-      scheme, _userinfo, _host, _port, _registry, _path, _opaque, _query, _fragment = URI.split(URI::DEFAULT_PARSER.escape(uri_str))
+      scheme, _userinfo, _host, _port, _registry, _path, _opaque, _query, _fragment = URI.split(URI::RFC2396_PARSER.escape(uri_str))
       scheme
     end
   end