-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Closed
Description
We currently buffer WebSocket opening until first LastHttpContent reception with the UpgradeCallback .
This doesn't make sense, and forces us to buffer any frame that might be sent along with the upgrade response.
WebSocketHandler and WebSocketUpgradeHandler need serious clean up:
- Drop
UpgradeHandlerthat's never used as an abstraction - Perform upgrade/abort as soon as response is received
- Ignore
LastHttpContent - No need to buffer any frame