-
Notifications
You must be signed in to change notification settings - Fork 58
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
side menu or log window closes after a few seconds #302
Comments
Hey @SemvdH , Thanks for taking the time to report this, I really appreciate it and well try to fix it for you if we can track down the bug together. Has this just started happening with a new update OliveTin, or are you a new user of OliveTin / it's always happened? Thanks for including the gif, but sadly it doesn't seem to load? |
@SemvdH , ah, I see the gif now, that is weirddd. What browser are you using there? I can see in the footer that the OliveTin websocket isn't connected - that could possibly be causing issues. Can you open your browser's web deveoper console and look for any errors? |
hi @jamesread , Thanks for the reply! I was using Chrome. I updated from an early 2023 version (I think march 2023) to the latest version. After updating I noticed the issue. After checking the console, I do indeed see some errors. There is an error message displayed with
There is also a It's probably the WSS GET request, could there be something wrong with my installation? The difference between the version I updated from and the version I am running now is pretty big |
Yeah, there have been quite a few changes between those versions - including adding websocket support. OliveTin is probably unable to connect back to itself on a websocket because of a reverse proxy configuration. Are you using a reverse proxy here? If so, could you dump your config? Note that the logs you posted include the full public URL to your OliveTin instance, so you may want to hide that / put some security in front of it. |
Okay, I can now see what's happening.
The fix plan is we should work out why your websocket isn't connecting (I bet that's a reverse proxy issue), but secondly, probably force-closing the sidebar is a bug if we fallback to this mode without websockets. |
I indeed use nginx as a reverse proxy, here's my config:
|
@SemvdH Yep, we need to add a couple of extra statements to that config now to handle websockets. It's documented here - https://docs.olivetin.app/nginx-dns.html - but this should be the only extra bit you need; location /websocket {
proxy_set_header Upgrade "websocket";
proxy_set_header Connection "upgrade";
proxy_pass http://localhost:1337/websocket;
} |
@jamesread That seems to have solved the issue, thanks a lot! Guess I should've checked the documentation a bit more... |
@SemvdH No problem, it's not a problem with you checking the documentation, it's just that a major new communication setting was added (websocket) which needed updated reverse proxy configs - seems like a weird reason for the sidebar to flash in and out! (I assume that is now fixed?) I'll keep this bug open, as I'll try to address the underlying cause of the sidebar flashing in and out. |
OliveTin now hard relies on websocket, and will display a big red error message if it isn't available. This means the the sidebar closing/opening is irrelevant, the websocket connection should be fixed. |
Describe the bug
When trying to view the side menu or the log window, after a few seconds the window closes and you go back to the buttons overview. This happens from about a second to a few seconds after opening the side menu. The hamburger goes back to the default state, but sometimes stays in the active state.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I expect the side menu and the log window to stay open.
Screenshots
bug gif
Additional context
Version: 2024.04.28 | Server connection: REST, WebSocket
OliveTin running on Ubuntu 22.04, installed through amd64 deb package
The text was updated successfully, but these errors were encountered: