From: Andrew Gerrand <adg@golang.org>
Date: Wed, 22 Sep 2010 06:25:26 +0000 (+1000)
Subject: http: escape _all_ reserved characters as per the RFC
X-Git-Tag: weekly.2010-09-22~9
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=d093bdbe7e10eb8b613b346ae5acba7569c97f7e;p=gostls13.git

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 136e6dfe4b..060619b835 100644
--- 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