http: escape _all_ reserved characters as per the RFC

R=r
CC=golang-dev
https://golang.org/cl/2199043

diff --git a/src/pkg/http/url.go b/src/pkg/http/url.go
index 136e6dfe4ba0e3aeaa381e9302890480818f1a83..060619b8355ec5aef2290a553a57a01dac40b7fa 100644 (file)
--- a/src/pkg/http/url.go
+++ b/src/pkg/http/url.go
@@ -61,7 +61,7 @@ func shouldEscape(c byte) bool {
        switch c {
        case '<', '>', '#', '%', '"', // RFC 2396 delims
                '{', '}', '|', '\\', '^', '[', ']', '`', // RFC2396 unwise
-               '?', '&', '=', '+': // RFC 2396 reserved in path
+               ';', '/', '?', ':', '@', '&', '=', '+', '$', ',': // RFC 2396 reserved
                return true
        }
        return false