-
Notifications
You must be signed in to change notification settings - Fork 400
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
Websocket connections fail on firefox #2191
Comments
@powertomato, which branch/version are you using? @pabera, is this
fixed by #2179? |
Firefox support in genereal should be fixed with the last release. Did you update to 3.5.0-develop or made a new installation? Can you share your jukebox.yaml and installation log? |
The correct version should be |
This shouldn't be a Firefox issue. It's usually a hint that the jukebox-daemon or the MPD is not running. It's what your stack trace also reports.
The issue is related to the recent PR #2177. Maybe there is a bug which we have not captured yet. Can you check if your |
Since I deleted the original installation log (sorry), I reran the setup-script. Here are the logs that resulted from that: INSTALL-1704151233.log All the Errors are now gone from the log. But the Firefox issue persists. Note: |
This was the issue. I removed pyzmq and rerun the setup, now the correct version was built and the webapp works in firefox too. So the only problem I see is, the setup should check for the correct pyzmq version, not only if pyzmq is installed. |
There are different problems here:
pyzmq
playermpd
|
Thats strange, as the entry is still missing in your
Good to know :) Strange it was a wrong version in the first place. Do you maybe have a second sd card to test a fresh installation?
Agree, we should evaluate if this can be easily tested. The problem is its not the pyzmq version itself, but the dependency to libzmq its build with. |
@pabera @AlvinSchiller Especially the cover art cache isn't something a user would need to customize, but usually could go with a default entry. In general this could help us to make the installation process more "updateable". |
Do we need to pin the versions of libzmq and pyzmq? |
I've done something similiar for the reader config, only there the check is for the file existing. Could also be possible for single entries?
|
libzmq cant be pinned, as a prebuild bundle is used or it needs to be build locally. And i think thats also true for pyzmq as it needs to be build against the local libzmq to support draft API. |
Yeah, something like that. In the past I had a project, which had a mechanism for creating default values, if the parameters or values weren't in the actual file. Made it also easier to always create a config file matching the code instead of making sure the shipped file matches the code. But this was for ini files :-/ |
Should be fixed with #2272 See release 3.5.2 |
Closing now as completed. Feel free to reopen, if the issue persists for you. |
Describe your problem
The UI only loads partially, dynamic values like volume, CPU temperature ect. display a permanent spinner-icon.
In the dev-console it is apparent that the connection to the websocket ws://192.168.1.222:5556 and *:5557, respectively (where the IP is the IP of the box)
Connection on same machine using chrome browser works.
The only difference I can spot is the header value
Sec-Websocket-Extensions
on the ws-request:Firefox:
Sec-WebSocket-Extensions: permessage-deflate
Chrome:
Sec-Websocket-Extensions: permessage-deflate; client_max_window_bits
What's your hardware set up?
Rock64 with DietPi
Log:
The text was updated successfully, but these errors were encountered: