http: do not parse req.URL for CONNECT

CONNECT's argument is not a URL.

R=golang-dev, bradfitz, rsc
CC=golang-dev
https://golang.org/cl/4808044

src/pkg/http/readrequest_test.go

@@ -152,6 +152,28 @@ var reqTests = []reqTest{
 		noBody,
 		"parse : empty url",
 	},
+
+	// CONNECT method.
+	{
+		"CONNECT proxy.example.com:443 HTTP/1.0\r\n" +
+			"Host: proxy.example.com\r\n\r\n",
+
+		&Request{
+			Method:        "CONNECT",
+			RawURL:        "proxy.example.com:443",
+			URL:           nil,
+			Proto:         "HTTP/1.0",
+			ProtoMajor:    1,
+			ProtoMinor:    0,
+			Close:         false,
+			ContentLength: 0,
+			Host:          "proxy.example.com",
+			Form:          Values{},
+		},
+
+		noBody,
+		noError,
+	},
 }
 
 func TestReadRequest(t *testing.T) {

src/pkg/http/request.go

@@ -548,9 +548,12 @@ func ReadRequest(b *bufio.Reader) (req *Request, err os.Error) {
 		return nil, &badStringError{"malformed HTTP version", req.Proto}
 	}
 
+	isConnect := strings.ToUpper(req.Method) == "CONNECT"
+	if !isConnect {
 		if req.URL, err = ParseRequestURL(req.RawURL); err != nil {
 			return nil, err
 		}
+	}
 
 	// Subsequent lines: Key: value.
 	mimeHeader, err := tp.ReadMIMEHeader()
@@ -566,7 +569,9 @@ func ReadRequest(b *bufio.Reader) (req *Request, err os.Error) {
 	//	GET http://www.google.com/index.html HTTP/1.1
 	//	Host: doesntmatter
 	// the same.  In the second case, any Host line is ignored.
+	if !isConnect {
 		req.Host = req.URL.Host
+	}
 	if req.Host == "" {
 		req.Host = req.Header.Get("Host")
 	}