net/http: redirect helper does not preserve querystring

[gopherbot]

by daniel.r.heath:

I'm using net/http ServeMux.

When I register '/foo/' as a route, then any attempt to GET '/foo' will redirect to
'/foo/'.

This is added in http://golang.org/src/pkg/net/http/server.go?s=25649:25677#962 .

However, any attempt to GET '/foo?query=string' will also redirect to '/foo'.

When we add redirection helpers, they shouldn't silently discard part of the request.

[minux]

Comment 1:

i created https://golang.org/cl/7099045/, but failed to mail it out.
@brad, is this issue worth Go1.1?

Labels changed: added priority-later, removed priority-triage.

Status changed to Accepted.

[bradfitz]

Comment 2:

What was the behavior in Go 1.0.3?

[gopherbot]

Comment 3 by daniel.r.heath:

This is current in both 1.0.3 and master (as of a week ago)

[bradfitz]

Comment 4:

If it's not a regression from Go 1.0.3, it's too late to fix it for Go 1.1.

[minux]

Comment 5:

Issue #5382 has been merged into this issue.

[rsc]

Comment 6:

Please fix for Go1.2 though.

Labels changed: added go1.2.

[bradfitz]

Comment 7:

Fixed in submitted https://golang.org/cl/7099045/ which didn't include a "Fixes
issue #5252" line.

[bradfitz]

Comment 8:

Status changed to Fixed.