Skip to content

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

Closed
mateusz-sterczewski wants to merge 1 commit into
mainfrom
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 into
mainfrom
CM-61550-handle-408-request-timeout

Conversation

@mateusz-sterczewski

Copy link
Copy Markdown
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>

@elsapet elsapet left a comment

Copy link
Copy Markdown
Collaborator

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
@doratias18
doratias18 deleted the CM-61550-handle-408-request-timeout branch August 26, 2026 20:25
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