]> Cypherpunks repositories - gostls13.git/commitdiff
net/http: remove redundant strings.TrimPrefix
authorErik Aigner <aigner.erik@gmail.com>
Mon, 2 Feb 2015 07:52:41 +0000 (08:52 +0100)
committerBrad Fitzpatrick <bradfitz@golang.org>
Tue, 3 Feb 2015 09:58:37 +0000 (09:58 +0000)
We already checked for the prefix with strings.HasPrefix

Change-Id: I33852fd19ffa92aa33b75b94b4bb505f4043a54a
Reviewed-on: https://go-review.googlesource.com/3691
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/net/http/request.go

index 487eebcb841beff999f0998942f3ff7a13c4e9ad..63d7d44aa0fb803fce4932c15b312ad877a8b4d9 100644 (file)
@@ -536,10 +536,11 @@ func (r *Request) BasicAuth() (username, password string, ok bool) {
 // parseBasicAuth parses an HTTP Basic Authentication string.
 // "Basic QWxhZGRpbjpvcGVuIHNlc2FtZQ==" returns ("Aladdin", "open sesame", true).
 func parseBasicAuth(auth string) (username, password string, ok bool) {
-       if !strings.HasPrefix(auth, "Basic ") {
+       const prefix = "Basic "
+       if !strings.HasPrefix(auth, prefix) {
                return
        }
-       c, err := base64.StdEncoding.DecodeString(strings.TrimPrefix(auth, "Basic "))
+       c, err := base64.StdEncoding.DecodeString(auth[len(prefix):])
        if err != nil {
                return
        }