Skip to content

Conversation

agentultra
Copy link

This change updates aiohttp to >= 0.22.1 and updates the code to the new
APIs.

This change updates aiohttp to >= 0.22.1 and updates the code to the new
APIs.


class HttpRequest(aiohttp.client.ClientRequest):
class HttpRequest(aiohttp.client_reqrep.ClientRequest):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use just aiohttp.ClientRequest

@kxepal
Copy link
Member

kxepal commented Aug 23, 2016

@agentultra Hi
Thanks for PR, but suddenly it requires a little bit more work to do. It's too long to explain, just hold on a bit. I hope to return to the project in a couple of days.

@agentultra
Copy link
Author

@kxepal Understood... some of the APIs have changed quite a bit and I did my best on the first pass to keep things lined up.

I'm working on a script that uses aiocouchdb with the aiobotocore library and they use conflicting versions of aiohttp. I'm here to help any way I can as I really need to get this working.

Following up on some review comments I changed the HttpRequest and
HttpResponse classes to subclass from aiohttp.ClientRequest and
aiohttp.ClientResponse respectively.

The lint check was catching an unused import in hdrs.py which I added a
skip to since it's importing names into the module that are used
elsewhere.
@SakuraSound
Copy link

Any chance of this getting done?

@kxepal
Copy link
Member

kxepal commented May 18, 2017

Oh, sorry! Yes, will handle this today.

@SakuraSound
Copy link

Thank you! I was originally going to fork and try to push something, but i saw that this was one Error shy of being merged.

@CLAassistant

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants