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

Websocket handling has changed #198

Closed
esticle opened this issue Sep 9, 2020 · 4 comments
Closed

Websocket handling has changed #198

esticle opened this issue Sep 9, 2020 · 4 comments

Comments

@esticle
Copy link

esticle commented Sep 9, 2020

Having recently upgraded from Swag 1.7.0 to 1.8.0 and re-rolling from the sample configs. I noticed that the removal of two lines in this commit has broken the websocket connection under 'Alerts' for a server.

A copied Plex sample config has had these two lines removed between v1.7.0 and 1.8.0:

proxy_set_header        Upgrade         $http_upgrade;
proxy_set_header        Connection      "upgrade";

If you are accessing your Plex instance via Swag/a configuration using these reverse-proxy-confs at version 1.8.0 you will notice that going to Settings -> Alerts you will get the text: Unable to listen for events on servername.

This is fixed by re-adding the above two lines.

Thanks.

@Roxedus
Copy link
Member

Roxedus commented Sep 9, 2020

How outdated is your nginx.conf and proxy.conf?

@esticle
Copy link
Author

esticle commented Sep 9, 2020

How outdated is your nginx.conf and proxy.conf?

Thanks for the hint - my nginx.conf was from Swag 1.7.0 which (of course) had this commit when it moved to 1.8.0 to sync with the reverse-proxy-confs removal of the same - which explains it all.

Know what to look for next time around now, thanks for the help!

@esticle esticle closed this as completed Sep 9, 2020
@nemchik nemchik pinned this issue Sep 10, 2020
@nemchik nemchik changed the title Plex websocket handling is incorrect, proposed fix Websocket handling has changed Sep 10, 2020
@nemchik
Copy link
Member

nemchik commented Sep 10, 2020

Ref: #201 #200 #199

@nemchik
Copy link
Member

nemchik commented Sep 10, 2020

For future users:

I've edited the subject, and in pinned this issue so it will be visible even while closed, and I'm locking discussion after posting this.

Proxy configs with web sockets are now handled differently. Users need to manually update their proxy.conf and nginx.conf. See the SWAG readme for more info. https://github.com/linuxserver/docker-swag#updating-configs

We do not need PRs to add back the web socket lines that were removed.

Thanks.

@linuxserver linuxserver locked as resolved and limited conversation to collaborators Sep 10, 2020
Roxedus referenced this issue Dec 30, 2020
Also general cleanup of whitespace
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants