Skip to content
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

libwebsockets: Ran out of header data space #9

Open
richdunajewski opened this issue Oct 15, 2020 · 2 comments
Open

libwebsockets: Ran out of header data space #9

richdunajewski opened this issue Oct 15, 2020 · 2 comments
Labels
question Further information is requested

Comments

@richdunajewski
Copy link

In the Janus log I see the following error for some clients:

[libwebsockets][ERR] Ran out of header data space

It looks like the issue is the cookies the browser (Chrome in this case) sends in the Websocket request for certain clients. I run several websites off my domain (i.e. the main website, a store, systems on subdomains) so there are cookies from Google, Facebook, Shopify, to name a few. If a client has visited these sites they will accumulate enough cookie data to exceed the default "max_http_header_data" value in libwebsockets.

This can be verified by using a client that hasn't accumulated a lot of cookies or by opening an Incognito window in Chrome which strips out the stored cookies.

How would we increase this limit? I believe it could/should be done in the Janus transport plugin, where it configures libwebsocket but there's also a way to change the default 4096 byte limit at build time of libwebsockets. Can that be done with this Docker image?

Other than this issue this image has been working perfectly, so thank you for your work!

@tranchitella
Copy link
Contributor

@richdunajewski updating the 4096 limit at buildtime of the libwebsockets is not doable as we are relying on the Debian package for the library. I believe the only way we can solve this issue is downloading and compiling our version of the library.

@tranchitella
Copy link
Contributor

@richdunajewski can you please retry with the latest master image?

@tranchitella tranchitella added the question Further information is requested label Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants