registryurl: deprecate GetHostname(), GetPort()

These were just wrappers for url.Hostname() and url.Port()

Signed-off-by: Sebastiaan van Stijn <[email protected]>

Author: thaJeztah

osxkeychain/osxkeychain_darwin.go

@@ -138,7 +138,7 @@ func (h Osxkeychain) List() (map[string]string, error) {
 	listLen = int(listLenC)
 	pathTmp := (*[1 << 30]*C.char)(unsafe.Pointer(pathsC))[:listLen:listLen]
 	acctTmp := (*[1 << 30]*C.char)(unsafe.Pointer(acctsC))[:listLen:listLen]
-	//taking the array of c strings into go while ignoring all the stuff irrelevant to credentials-helper
+	// taking the array of c strings into go while ignoring all the stuff irrelevant to credentials-helper
 	resp := make(map[string]string)
 	for i := 0; i < listLen; i++ {
 		if C.GoString(pathTmp[i]) == "0" {
@@ -160,7 +160,7 @@ func splitServer(serverURL string) (*C.struct_Server, error) {
 		proto = C.kSecProtocolTypeHTTP
 	}
 	var port int
-	p := registryurl.GetPort(u)
+	p := u.Port()
 	if p != "" {
 		port, err = strconv.Atoi(p)
 		if err != nil {
@@ -170,7 +170,7 @@ func splitServer(serverURL string) (*C.struct_Server, error) {
 
 	return &C.struct_Server{
 		proto: C.SecProtocolType(proto),
-		host:  C.CString(registryurl.GetHostname(u)),
+		host:  C.CString(u.Hostname()),
 		port:  C.uint(port),
 		path:  C.CString(u.Path),
 	}, nil

registryurl/parse.go

@@ -28,7 +28,7 @@ func Parse(registryURL string) (*url.URL, error) {
 		return nil, errors.New("unsupported scheme: " + u.Scheme)
 	}
 
-	if GetHostname(u) == "" {
+	if u.Hostname() == "" {
 		return nil, errors.New("no hostname in URL")
 	}
 
@@ -37,11 +37,15 @@ func Parse(registryURL string) (*url.URL, error) {
 }
 
 // GetHostname returns the hostname of the URL
+//
+// Deprecated: use url.Hostname()
 func GetHostname(u *url.URL) string {
 	return u.Hostname()
 }
 
 // GetPort returns the port number of the URL
+//
+// Deprecated: use url.Port()
 func GetPort(u *url.URL) string {
 	return u.Port()
 }