Skip to content

Socket Read Timeout remove validation#1417

Merged
scottf merged 2 commits intomainfrom
scoket-read-timeout
Sep 11, 2025
Merged

Socket Read Timeout remove validation#1417
scottf merged 2 commits intomainfrom
scoket-read-timeout

Conversation

@scottf
Copy link
Contributor

@scottf scottf commented Sep 11, 2025

  • The validation was wrong to begin with
  • This is a potential footgun if set too low

* The validation was wrong to begin with
* This is a potential footgun if set too low
Copy link
Member

@mtmk mtmk left a comment

Choose a reason for hiding this comment

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

LGTM

@scottf scottf merged commit ec624ef into main Sep 11, 2025
5 checks passed
@scottf scottf deleted the scoket-read-timeout branch September 11, 2025 15:48
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

Successfully merging this pull request may close these issues.

2 participants