Use standard library redaction functionality

http/url/url.go

@@ -56,13 +56,12 @@ func AddBasicAuth(joinAddr, username, password string) (string, error) {
 	return u.String(), nil
 }
 
-// RemoveBasicAuth returns a copy of the given URL, with any basic auth information
+// RemoveBasicAuth returns a copy of the given URL, with any basic auth password
-// removed.
+// redacted.
 func RemoveBasicAuth(u string) string {
 	uu, err := url.Parse(u)
 	if err != nil {
 		return u
 	}
-	uu.User = nil
+	return uu.Redacted()
-	return uu.String()
 }

http/url/url_test.go

@@ -121,15 +121,15 @@ func Test_RemoveBasicAuth(t *testing.T) {
 		},
 		{
 			orig:    "https://foo:bar@localhost",
-			removed: "https://localhost",
+			removed: "https://foo:xxxxx@localhost",
 		},
 		{
 			orig:    "https://foo:bar@localhost:4001",
-			removed: "https://localhost:4001",
+			removed: "https://foo:xxxxx@localhost:4001",
 		},
 		{
 			orig:    "http://foo:bar@localhost:4001/path",
-			removed: "http://localhost:4001/path",
+			removed: "http://foo:xxxxx@localhost:4001/path",
 		},
 	}