]> Cypherpunks repositories - gostls13.git/commit
net/url: allow all valid host chars in RawPath
authorRuss Cox <rsc@golang.org>
Thu, 6 Aug 2015 02:12:16 +0000 (22:12 -0400)
committerRuss Cox <rsc@golang.org>
Thu, 6 Aug 2015 02:59:16 +0000 (02:59 +0000)
commitfced03a5c6dd22dd486106e3dd116510c28c6e4a
treea5bcc2f104eb3da8ccacd905b38dc1a8dc72ad0a
parente8be9a170c3044d7460a6b2c8349a723b1a21dd2
net/url: allow all valid host chars in RawPath

The old code was only allowing the chars we choose not to escape.
We sometimes prefer to escape chars that do not strictly need it.
Allowing those to be used in RawPath lets people override that
preference, which is in fact the whole point of RawPath (new in Go 1.5).

While we are here, also allow [ ] in RawPath.
This is not strictly spec-compliant, but it is what modern browers
do and what at least some people expect, and the [ ] do not cause
any ambiguity (the usual reason they would be escaped, as they are
part of the RFC gen-delims class).
The argument for allowing them now instead of waiting until Go 1.6
is that this way RawPath has one fixed meaning at the time it is
introduced, that we should not need to change or expand.

Fixes #5684.

Change-Id: If9c82a18f522d7ee1d10310a22821ada9286ee5c
Reviewed-on: https://go-review.googlesource.com/13258
Reviewed-by: Andrew Gerrand <adg@golang.org>
src/net/url/url.go
src/net/url/url_test.go