Remove digest_transport.go wrapper

Reference: #4599 (review)
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>

Author: Juneezee

lib/netext/httpext/digest_transport.go

@@ -14,6 +14,7 @@ import (
 	"time"
 
 	"github.com/Azure/go-ntlmssp"
+	"github.com/icholy/digest"
 	"github.com/sirupsen/logrus"
 	"gopkg.in/guregu/null.v3"
 
@@ -213,7 +214,19 @@ func MakeRequest(ctx context.Context, state *lib.State, preq *ParsedHTTPRequest)
 				return status == 401 || originalResponseCallback(status)
 			}
 		}
-		transport = digestTransport{originalTransport: transport}
+
+		username := preq.URL.u.User.Username()
+		password, _ := preq.URL.u.User.Password()
+
+		// Remove the user data from the URL to avoid sending the authorization
+		// header for basic auth
+		preq.URL.u.User = nil
+
+		transport = &digest.Transport{
+			Username:  username,
+			Password:  password,
+			Transport: transport,
+		}
 	} else if preq.Auth == "ntlm" {
 		// The first response of NTLM auth may be a 401 error.
 		if tracerTransport.responseCallback != nil {