-
-
Notifications
You must be signed in to change notification settings - Fork 356
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
add "social" login #301
Comments
Perhaps using http://www.keycloak.org? |
Not sure if 'social login' feature solves a lot, well depending on what sort of users? For owners it doesn't solve a lot, there is no Organisation etc. Perhaps it would be nice for the ticket buyers, to speed up the purchase process? In my case I need authorisation and authentication against my 3rd party platform, where there are: 'admins', 'owners', 'users'.
|
@kivanov82 Yes, you're right. Perhaps "SSO using 3rd party services" would be more appropriate as title?
If the subsystem can be enabled by the admin after the first login (which should be done using the auto-generated username/password), then we could let him/her define a whitelist of authorized email / domains.
Please note that currently alf.io doesn't manage (by design) the identity of the ticket buyer. Just to be on the same page: what @syjer and I had in mind was adding another Spring Security handler for 3rd party authentication. This would require an update to Spring 5.x and Spring Boot 2.x, which we'll do as soon as we start the actual work on v2.
in the blog post above, the actual authentication (either form-based or "social") is done on KeyCloak. Would this work also in your use case, or do you strictly require SAML-like authentication?
Would you enable this only for requests coming from the mobile app? |
@cbellone I'm guessing one could even do SAML-like authentication through KeyCloak. My Mr. KeyCloak colleague @sebastienblanc may like to chime in (Seb, cool project here). |
@vorburger I know this project and I know @cbellone , we chatted at Devoxx Morocco ;) (World is small) |
implemented in: #881 . |
The text was updated successfully, but these errors were encountered: