Skip to content

Improve error message for proxy connection failures #155

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hsenag
Copy link
Member

@hsenag hsenag commented Apr 30, 2024

Note - this is only for me to play with, I opened it with Copilot Workspace without looking closely at what it did. I'm not expecting anyone else to review it in its current form.

Related to #84

Implements enhanced error handling for proxy connection failures across the HTTP library, specifically targeting scenarios where the proxy mentioned in http_proxy is down.

  • Introduces a new error type ErrorProxyConnection in Network/Stream.hs to specifically handle proxy connection errors, improving error specificity and clarity for such issues.
  • Updates error handling in Network/StreamSocket.hs to detect and report proxy connection failures, utilizing the new ErrorProxyConnection error type. This includes handling specific socket error codes related to proxy connection failures.
  • Modifies Network/HTTP.hs to include the updated error handling mechanisms from Network/Stream.hs and Network/StreamSocket.hs, ensuring that proxy connection errors are correctly reported and logged, enhancing the overall robustness of the library in handling network-related errors.

For more details, open the Copilot Workspace session.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant