canyanio / janus-gateway-docker

Docker image for the Janus WebRTC Server
Other
160 stars 64 forks source link

libwebsockets: Ran out of header data space #9

Open richdunajewski opened 3 years ago

richdunajewski commented 3 years ago

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 commented 3 years ago

@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 commented 3 years ago

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