Attach Request and Response to HttpProcessingError raised from client

[justanr]

With request's Response.raise_for_status, the raised exception has the request and responses that caused that status attached to it. This allows handling of error codes + content to be off loaded from the calling code (into a decorator etc).

If there's interest, I'm happy to put together a PR.

[fafhrd91]

could you provide PR in this repo https://github.com/aio-libs/aiohttp

[justanr]

Yeah, is that the new canonical repo?

[fafhrd91]

I dont really want to document ClientRequest class, too much specific low level details. otherwise any PRs are welcome

[penguinolog]

Response to HttpProcessingError - i agree this idea. if @fafhrd91 @asvetlov will accept - pull request will be made at short time.
Currently, on our deploy, we to use our own raise_for_status with this functionality.

[fafhrd91]

ClientResponseError now has request_info attribute

[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.