-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
feat(proxyd): betterer timeoutz #6483
Conversation
|
Is there any way we could get some automated tests for these cases? |
added |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #6483 +/- ##
===========================================
+ Coverage 40.85% 44.26% +3.41%
===========================================
Files 393 315 -78
Lines 32334 25106 -7228
Branches 1689 0 -1689
===========================================
- Hits 13211 11114 -2097
+ Misses 17682 12978 -4704
+ Partials 1441 1014 -427
Flags with carried forward coverage won't be shown. Click here to find out more. |
28a6a95
to
c55edb5
Compare
c55edb5
to
428c086
Compare
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
This change refactor timeouts.
It also introduces new WS read and write timeouts; and enforces a body size limit for WS conns.
Tests
Defaults
Fixes https://linear.app/optimism/issue/INF-380/proxyd-read-limit-and-timeouts