-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Cannot change color theme on prelease version 4.0.0 #4642
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
Comments
I'm also adding here the logs from
|
Hmm... when I do it from the Command Palette, it appears to work. VideoScreen.Recording.2021-12-16.at.5.03.20.PM.mov
Where is this exactly? I may have hidden it. Do you mind posting a screenshot or video? |
color_test_code_server.movTo be honest I'm not sure exactly how you do it directly from the dropmenu in the bar. I doesn't work for me that way. I've just tested the app directly and it works - so without the reverse proxy (which was of course always a possibility given existence of the port in the url). Now the difference is (when accesing it over the proxy) that on the lower left-hand side of the screen where the fqdn is displayed, I see the port also ( As I said, I'm not enabling https on the code-server level, I'm providing this with the proxy. I'm guessing there must be a way of solving this with nginx, although, as I previously said, P.S. In my case code-server listens on port 8888, so I'm not even sure where it gets port 80 at all - that's why I'm inclined to believe the port is more or less hardcoded there. |
Related to #4608. This should be resolved with the next iteration of the 4.0.0 milestone. |
@lethargosapatheia thank you for the detailed explanation and the video! As @benz0li said (thank you!), this should be fixed in the next version. |
Is it okay if we close this? |
If this question if direct to me, yes, sure! I'm glad I could contribute a little bit :) |
Yes! (sorry should have @'ed you to be clear :) ) Thanks for helping test 4.0.0 and make sure it's solid for release! |
OS/Web Information
code-server --version
: 4.0.0Steps to Reproduce
Expected
Color theme should change.
Actual
Changing the color theme is simply ignored.
Logs
Logs from journalctl:
More relevant info from dev tools in chrome where I get
failed
(network tab):In the console:
Screenshot
Accessing several random color themes:

Notes
As far as I can see this seems to be a classical issue with trying to access https on port 80. I'm not sure what has changed in this version (I'm guessing a hardcoded port (80), because vscode itself is not configured for https, as I'm accessing it through a reverse proxy - nginx-controller in kubernetes).
Downgrading to 3.12.0 works instantly.
I should also mention that
port_in_redirect
is off (default behaviour for nginx controller).The text was updated successfully, but these errors were encountered: