Validate headers specific to upgrade request only when response is 101. - #113
Conversation
Call listener_factory before validating upgrade specific headers
Merging this PR will improve performance by 14.07%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ⚡ | test_bench_echo[32768] |
2 s | 1.7 s | +14.07% |
Tip
Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.
Comparing custom_http_responses (3254076) with master (bbea420)
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #113 +/- ##
==========================================
+ Coverage 91.00% 91.78% +0.78%
==========================================
Files 18 18
Lines 2590 2667 +77
Branches 138 139 +1
==========================================
+ Hits 2357 2448 +91
+ Misses 190 176 -14
Partials 43 43 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
The parser now enforces:
WebSocket validation has direct unit coverage for every existing branch, including accept-value calculation and non-ASCII keys. |
|
Closes #112 |
No description provided.