REQUEST_URI should preserve the query string

[fmang]

REQUEST_URI was introduced by #1419: 7162f43

Currently, Werkzeug strips the query string. While REQUEST_URI is not a standard variable, I believe the consensus is to include the query string. According to #766 (comment) that should have be the case for Werkzeug too.

That applies to RAW_URI too.

It’s an edge case, but ideally empty query strings like in /? should be preserved too and not simplified to /.

[davidism]

I can't reproduce this issue. I ran the following example and got the full unquoted path and query.

from pprint import pformat
from werkzeug import Request, Response, run_simple

@Request.application
def app(request):
    return Response(pformat(request.environ))

if __name__ == "__main__":
    run_simple("localhost", 5000, app)

Navigating to http://localhost:5000/a%2fb?a=%2f, REQUEST_URI was /a%2fb?a=%2f.

[davidism]

#1782 shows that this is about the test client, not the dev server.

[kx-chen]

Working on this.