Move URI::HTTP#authority to URI::Generic

The concept of authority is not specific to http/https URIs, so
URI::Generic is more appropriate place for this method.

This also fixes the missing userinfo part in the authority.

Author: rhenium

lib/uri/generic.rb

@@ -747,6 +747,45 @@ def port=(v)
       port
     end
 
+    #
+    # == Description
+    #
+    # Returns the authority of the URI, as defined in
+    # https://www.rfc-editor.org/rfc/rfc3986#section-3.2.
+    #
+    #   authority = [ userinfo "@" ] host [ ":" port ]
+    #
+    # Returns an empty string if no authority is present.
+    #
+    # == Usage
+    #
+    #   require 'uri'
+    #
+    #   URI::HTTP.build(host: 'www.example.com', path: '/foo/bar').authority
+    #   #=> "www.example.com"
+    #   URI::HTTP.build(host: 'www.example.com', port: 8000, path: '/foo/bar').authority
+    #   #=> "www.example.com:8000"
+    #   URI::HTTP.build(host: 'www.example.com', port: 80, path: '/foo/bar').authority
+    #   #=> "www.example.com"
+    #   URI::HTTP.build(host: 'www.example.com', userinfo: 'user:password').authority
+    #   #=> "user:password@www.example.com"
+    #
+    def authority
+      str = ''.dup
+      if self.userinfo
+        str << self.userinfo
+        str << '@'
+      end
+      if @host
+        str << @host
+      end
+      if @port && @port != self.default_port
+        str << ':'
+        str << @port.to_s
+      end
+      str
+    end
+
     def check_registry(v) # :nodoc:
       raise InvalidURIError, "cannot set registry"
     end
@@ -1365,17 +1404,7 @@ def to_s
         if @host || %w[file postgres].include?(@scheme)
           str << '//'
         end
-        if self.userinfo
-          str << self.userinfo
-          str << '@'
-        end
-        if @host
-          str << @host
-        end
-        if @port && @port != self.default_port
-          str << ':'
-          str << @port.to_s
-        end
+        str << self.authority
         if (@host || @port) && !@path.empty? && !@path.start_with?('/')
           str << '/'
         end

lib/uri/http.rb

@@ -93,27 +93,6 @@ def request_uri
       url.start_with?(?/.freeze) ? url : ?/ + url
     end
 
-    #
-    # == Description
-    #
-    # Returns the authority for an HTTP uri, as defined in
-    # https://www.rfc-editor.org/rfc/rfc3986#section-3.2.
-    #
-    #
-    # Example:
-    #
-    #     URI::HTTP.build(host: 'www.example.com', path: '/foo/bar').authority #=> "www.example.com"
-    #     URI::HTTP.build(host: 'www.example.com', port: 8000, path: '/foo/bar').authority #=> "www.example.com:8000"
-    #     URI::HTTP.build(host: 'www.example.com', port: 80, path: '/foo/bar').authority #=> "www.example.com"
-    #
-    def authority
-      if port == default_port
-        host
-      else
-        "#{host}:#{port}"
-      end
-    end
-
     #
     # == Description
     #

test/uri/test_http.rb

@@ -73,9 +73,9 @@ def test_authority
     assert_equal('a.b.c', URI.parse('http://a.b.c/').authority)
     assert_equal('a.b.c:8081', URI.parse('http://a.b.c:8081/').authority)
     assert_equal('a.b.c', URI.parse('http://a.b.c:80/').authority)
+    assert_equal('userinfo@a.b.c', URI.parse('https://userinfo@a.b.c/').authority)
   end
 
-
   def test_origin
     assert_equal('http://a.b.c', URI.parse('http://a.b.c/').origin)
     assert_equal('http://a.b.c:8081', URI.parse('http://a.b.c:8081/').origin)