-
Notifications
You must be signed in to change notification settings - Fork 14
Automatically close login webpage or redirect back to the web ide, or just give a notification #97
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
Quite funny, we we're literally just talking about that feature in the team. I'll assign this to the respective contributor :) |
Should be quite easy to implement :) |
Hi, maybe I don't fully understand your usecase here, but there is already a way to redirect back to the ide. At least for the oauth endpoints you can add a |
thinking more about this. One thing that would be nice could be to have a full login flow without having to open a new tab or popup. So that at the end you would be redirected back to the webide with the login token as url search paramter. So the application could then when the token paramter is available just send the login successful check request once and then have the user info available to create or join a session. |
@jonah-iden I've also recently thought that we could improve the login quite a bit - for example, the whole unverified/anonymous login flow can be done without leaving VS Code, just by entering data into the quick pick. Other providers would still require users to leave VS Code and open a browser, but that could redirect to the |
@msujew the problem with showing a quick pick is that most of the configuration on how to do the login is currently on server side. We would need some way from the server to request login methods. But that would then make the whole login flow more complicated. So
|
@jonah-iden I would argue that, since the anonymous login is essentially hardcoded into the server, we could have special handling for it, whereas everything else just delegates to the browser. That should be manageable, right? |
@msujew you would need to check in the client if it is enabled in the server and then offer the user a choice if other methods are available. Currently this choosing is basicly handled by the https://www.open-collab.tools/login/ website |
Yes, this is about simple login. Or can I use custom OAuth in offline deployment of oct-server? I mean current simple login in self-deploy environment would just leave user in the login web page after they clicked |
@ParticleG ah ok, thanks for the explanation. You are using the vscode extension right? |
Yes, I'm using the vscode extension in the code server (A web-based vscode compatible IDE) in a docker container. Thanks for making the PR. Using quick picks and ensuring all simple login steps are within the vscode is good. Allowing users to use a custom website is also appreciated. Adding a simple message to the current default simple login website sounds essential to me. We could also call |
Currently there's no notification indicates that user is logged in (authorized), neither having any automatic way to close login page or redirect to web ide.
The text was updated successfully, but these errors were encountered: