From d093bdbe7e10eb8b613b346ae5acba7569c97f7e Mon Sep 17 00:00:00 2001
From: Andrew Gerrand <adg@golang.org>
Date: Wed, 22 Sep 2010 16:25:26 +1000
Subject: [PATCH] http: escape _all_ reserved characters as per the RFC

R=r
CC=golang-dev
https://golang.org/cl/2199043
---
 src/pkg/http/url.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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
-- 
2.48.1