Remove fragment string in request path

[carlcarl]

What these changes does?

It remove the fragment string begin with # in request.path.
This should not be sent to the server with the url

The request.url still keeps the original format.

How to test your changes?

Sample code:

import asyncio
import aiohttp

async def test_request():
    with aiohttp.ClientSession() as session:
        async with session.get('http://127.0.0.1:8000/test#123') as resp:
            print(resp)

loop = asyncio.get_event_loop()
loop.run_until_complete(test_request())

1. Run a local server first: python3 -m http.server
2. Run the sample code above.
3. Check the message of the server, original will be 127.0.0.1 - - ... "GET /test#123 HTTP/1.1" 404 -, fixed will be 127.0.0.1 - - ... "GET /test HTTP/1.1" 404 -

Related issue number

#824

Checklist

• Code is written and well
• Tests for the changes are provided
• Documentation reflects the changes

[carlcarl]

Any suggestion?

[asvetlov]

@carlcarl sorry, you PR was closed accidentally by base branch removing.
I cannot reopen the PR but merged it manually by 01057dc.
Thank you very much for contribution.

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.