-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add configuration for WebSocket MaxFrameSize and MaxMessageSize #43435
Conversation
dometec
commented
Sep 23, 2024
•
edited by geoand
Loading
edited by geoand
- Fixes: Max frame length of 65536 has been exceeded - Quarkus 3 #43381
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense to add these there, but we need to think how we can advertize them from web-socket-next.
@mkouba WDYT?
(why it makes sense: because someone can use websockets directly from a router or reactive routes, it does not need to use Undertow websocket or websocket next).
Status for workflow
|
🎊 PR Preview ffd0c78 has been successfully built and deployed to https://quarkus-pr-main-43435-preview.surge.sh/version/main/guides/
|
We could just a section/note in the reference guide.
Well, it's possible but IMO not very common. |
The Dev UI uses a web socket. So, these configurations will affect it. It's not common, but we do use that approach. |
Status for workflow
|
Thanks! |