calzoneman / sync

Node.JS Server and JavaScript/HTML Client for synchronizing online media
Other
1.45k stars 235 forks source link

No registered users in connected users panel #969

Closed pojlFDlxCOvZ4Kg8y1l4 closed 1 year ago

pojlFDlxCOvZ4Kg8y1l4 commented 1 year ago

Please fill out the templates below to the best of your ability, based on whether your problem is with using the website or with running your own server.

Website Problem

Please confirm whether you've tried the following debugging steps:

Description of the Problem

System Information

image

image

calzoneman commented 1 year ago

From your first screenshot, it seems like the userlist is loading correctly, it's just that your browser is not being authenticated as your logged in account -- the behavior you described matches with how the page would appear to a logged-out user.

  1. Can you confirm that you are able to use other logged-in functionality, and it's just the channel that isn't authenticating you? For example, do you see the Account dropdown on the navbar, and are you able to access those pages normally? If you log out and back in (or log in from a private browser window), then does it work?
  2. Can you confirm that when you upgraded your server, you followed any relevant items in the NEWS.md and ran npm install to rebuild the application and its dependencies?
  3. Can you try clearing your browser cache and if relevant any website caches as well (e.g. if you are using Cloudflare, clear its cache)? In the second screenshot, there is an error printed about trying to access a variable that doesn't exist, which may not directly be related to your problem, but it does indicate that you have mismatched versions of the client scripts -- videojs.options.flash is no longer referenced anywhere in player.js
pojlFDlxCOvZ4Kg8y1l4 commented 1 year ago
  1. I am able to use logged in functionality: image But after logging out and back in, restarting browser, cache wipe etc. it still does not show my user in the left pane: image

  2. history shows following commands were used: 285 git pull 286 npm install 287 npm run postinstall But I did not see NEWS.md. I will look through that and will report back later today.

  3. It may or may not be that Cloudflare tunnel is fiddling with the scripts. Development mode makes the error go away, it seems. I will come back and confirm this later today.

calzoneman commented 1 year ago

It may or may not be that Cloudflare tunnel is fiddling with the scripts. Development mode makes the error go away, it seems. I will come back and confirm this later today.

If I recall, development mode disables all caching. So you may just need to go to Caching > Configuration and purge the cache.

pojlFDlxCOvZ4Kg8y1l4 commented 1 year ago

If I recall, development mode disables all caching. So you may just need to go to Caching > Configuration and purge the cache.

Cache wipes do not work. I've dug around and tested some stuff, it seems Cloudflare's Rocket Loader does some weird stuff. Cloudflare's Developer mode enabled bypasses cache and with that Rocket Loader completely. So I've fiddled with Rocket Loader. With Rocket Loader enabled, it wants to use the deprecated videojs.options.flash. With Rocket Loader disabled, it shows the connected users in the left pane.
I've observed similar odd behaviour with Nextcloud, which also has Rocket Loader disabled.

Globally rocket loader is still enabled for the domain, so other sites can use it. I've created a page rule for Cytube, with following settings: URL: subdomain.domain.tld/* Pick setting: Rocket loader Toggle set to disabled, (x) image

I'll try it out in the next few days, if this solves the problem then I think it's safe to say it's just Cloudflares Rocket Loader doing it's oddities once again.

calzoneman commented 1 year ago

Closing as per your comment it appears to work when rocket loader is disabled thus it seems to be a server configuration issue and not a code issue. Feel free to reopen if there is new evidence that there may be a bug in the software.