-
Notifications
You must be signed in to change notification settings - Fork 5.9k
v3.1.0 doesn't work for me - Can't log in #1507
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
Same here for me. Just use the older 3.0.2 version and it works. Yet ignore update to 3.1.0 notification until this is fixed |
Hm will definitely fix, sorry about this. Looking into it. cc: @code-asher |
I'm definitely willing to help reproduce and debug. I'm guessing it works on your machine, else you wouldn't have pushed it. |
Hmm, so far I haven't been able to replicate. It should only redirect when it doesn't find the correct key in the cookies so maybe something is wonky there. Could you check the Chrome dev tools and see what the For example when I log in I get a single |
Mostly just looking to see if you have more than one cookie or if the domain or path doesn't match the URL you're browsing. |
When I run on clean download with only |
Oh hmm one thing that changed in 3.1.0 was that we added the domain to the cookie. It'll use the Does your nginx configuration do anything interesting with the host header, maybe? |
Can't check right now, I'll get back to you tomorrow. I think it only uses |
I have mine set to:
I just tested without setting a host header and now I can't log in. That seems like a bug so I'll look into it. |
Sorted it out; thanks for the bug report! The current workaround until the next version comes out with the fix is to set the host header in the proxy config. |
Thanks so much for the support and quick fix! I'll just skip that version and go straight to the next one, when it comes out . |
I also had this same issue, I thought that I needed to pass a proxy-domain argument into code-server to fix the login issue. This seemed to do the trick though! |
Oh, yeah, I can see how that flag would be confusing! For anyone that's curious, the I've just tagged 3.1.1 so we'll have a release with this fix out soon so the host header will no longer be required for logging in. Although it's probably not a bad idea to forward the host information anyway. :) |
Hi, I've just updated from an older 2.x version to v3.1.0 but I cannot login. It keeps redirecting me from
/
to/login?to=%2F
.I downgraded to v3.0.2 and that works fine.
Only lead I can give is that this error is logged in chrome:
The text was updated successfully, but these errors were encountered: