Skip to content
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

[feat] multiple idp : add a "remember my choice" option #795

Open
CorentinS6 opened this issue Dec 8, 2023 · 2 comments
Open

[feat] multiple idp : add a "remember my choice" option #795

CorentinS6 opened this issue Dec 8, 2023 · 2 comments

Comments

@CorentinS6
Copy link

Hello,
In case of multiple idp, nextcloud display the list, and the user has to choose.
It would be interesting to add an option to save the choice, and no longer offer the list at the next connection (and redirect the user directly to the memorized idp).
Best regards.

@blizzz
Copy link
Member

blizzz commented Jan 16, 2024

Hi @CorentinS6, thanks for pointing it out. I think it makes a lot of sense to have the last entry pre-selected. I am doubtful about an automatic redirect, as it would be difficult to return to the picker again.

Would you perhaps be up for implementing this? Or would anyone else be up for implementing it? I am happy to review any PR, or help getting unstuck in case of difficulties.

@CorentinS6
Copy link
Author

Hi,
I can try to do this. We can first discuss how it might work.
My first naive idea : on login success, set a cookie to remeber idp. on next authentification, pre-select the same idp with a (visual) JS timer before the redirect (like grub on system boot).
do you have any other ideas?
regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants