Skip to content

CM-61550: Handle HTTP 408 with a user-friendly slow connection message#417

Closed
mateusz-sterczewski wants to merge 1 commit intomainfrom
CM-61550-handle-408-request-timeout
Closed

CM-61550: Handle HTTP 408 with a user-friendly slow connection message#417
mateusz-sterczewski wants to merge 1 commit intomainfrom
CM-61550-handle-408-request-timeout

Conversation

@mateusz-sterczewski
Copy link
Collaborator

Adds HttpRequestTimeoutError as a distinct subclass of RequestHttpError for 408 responses, so users see a specific message about slow connections instead of the generic "unable to complete scan" error.

Adds HttpRequestTimeoutError as a distinct subclass of RequestHttpError
for 408 responses, so users see a specific message about slow connections
instead of the generic "unable to complete scan" error.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Collaborator

@elsapet elsapet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 😎

@gotbadger gotbadger added the do not merge Used when a PR has been approved but we don't want to merge just yet label Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Used when a PR has been approved but we don't want to merge just yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants