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

Make HttpRequestException inherit from IOException #847

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

jan-tennert
Copy link
Collaborator

What kind of change does this PR introduce?

Feature/Fix (closes #846)

What is the current behavior?

HttpRequestException is a "normal" exception, in the contrary to the HttpRequestTimeoutException which inherits from IOException

What is the new behavior?

HttpRequestException now inherits from IOException

@jan-tennert jan-tennert added the enhancement New feature or request label Feb 2, 2025
@jan-tennert jan-tennert self-assigned this Feb 2, 2025
@jan-tennert jan-tennert merged commit 59beb73 into master Feb 4, 2025
13 checks passed
@jan-tennert jan-tennert deleted the io-exception branch February 4, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Question]: Shouldn't HttpRequestException subclass IOException?
1 participant