Add ClientResponse().history elements to ClientResponseError

[penguinolog]

Long story short

During investigation of HTTP client exceptions on big installations, history can provide information about failure reason (incorrect redirect and etc).
De-facto needed fields:
url, code, reason, location if present in headers (all headers is better)

Expected behaviour

Accessible list ClientResponseError().history

Actual behaviour

Currently this information is unavailable.

Proposed solution:

HistoryRecord = namedtuple('HistoryRecord', ('url', 'code', 'reason', 'headers'))
ClientResponseError().history: typing.List[HistoryRecord]

[fafhrd91]

Let's just add history to exception. Do we really need new type?

[penguinolog]

Just following request_info idea.
May I add just response to exception and expose it fields? (like it done in requests).

[fafhrd91]

request_info is required only because ClientRequest is private class.
ClientResponseError may be raised without response, so exception should have history of interactions and response that is cause of exception or none if there is no one.

[penguinolog]

Accepted. Thank you!

[fafhrd91]

Merged

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