http: escape _all_ reserved characters as per the RFC

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

http: escape _all_ reserved characters as per the RFC
R=r CC=golang-dev https://golang.org/cl/2199043
d093bdbe · Andrew Gerrand · 22974fbe · d093bdbe
Commit d093bdbe authored Sep 22, 2010 by Andrew Gerrand
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

src/pkg/http/url.go src/pkg/http/url.go +1 -1

No files found.
--- 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