Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reqwest::Error now less informative #2210

Closed
Smotrov opened this issue Mar 26, 2024 · 1 comment
Closed

reqwest::Error now less informative #2210

Smotrov opened this issue Mar 26, 2024 · 1 comment

Comments

@Smotrov
Copy link

Smotrov commented Mar 26, 2024

With updated version to_string() trait method of reqwest::Error is now not the informative when in case when there is a connection error. It is always only error sending request now.
Before it was something like error sending request: operation timed out,
error sending request: connection closed before message completed, error sending request: connection error: Connection reset by peer (os error 104) etc.
Now it is much harder to investigate what is causing the issues. Would be grate to bring back previous descriptions.

@seanmonstar
Copy link
Owner

I know, it's not ideal. But it's also expected by a lot of other people: hyperium/hyper#2732

The idea is that you would use some sort of error reporter (such as the eventually stable std::error::Report) that will print each error message, following it's source chain. If each error printed the chain itself, you'd have a lot of duplication.

@seanmonstar seanmonstar closed this as not planned Won't fix, can't repro, duplicate, stale Mar 26, 2024
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

No branches or pull requests

2 participants