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

Integrare SSO #26

Open
alexchiraples opened this issue Aug 6, 2017 · 0 comments
Open

Integrare SSO #26

alexchiraples opened this issue Aug 6, 2017 · 0 comments

Comments

@alexchiraples
Copy link

alexchiraples commented Aug 6, 2017


Migrated from https://github.com/gov-ithub/volunt-app#58
Originally created by @rusanu anu on Sun, 06 Aug 2017 16:49:17 GMT


https://github.com/civictechro/auth-sso

Sa permitem utilizatorilor sa foloseasca credentiale unice pe siturile gov.ro. Integrarea cu vlunt-app se doreste si o demonstratie tehnica de integrare SSO gov.ro cu o aplicatie RoR. Intrucit in RoR se foloseste mult Devise, ar fi preferabil ca solutia noastra sa demonstreze cum se integreaza SSO cu Devise, ca un OAuth provider. Integrarea cere probabil un gem separat de Rail care sa poata fi re-folosit in alte proiecte. Ideal acest gem ar trebuii sa ofere OAuth cu gov.ro atit prin Devise cit si standalone.

SSO trebuie oferit ca o alternativa la login. Login user/password trebuie sa ramina in continuare valid pentru ca 1) dependinta de SSO fara fall-back poate crea probleme in aplicatie daca avem problme cu SSO si 2) exista un numar mare de utilizatori in volunt-app care probabil nu vor dori sa fie fortati sa migreze.

Exista mai multe probleme cu volunt-app login care ar trebuii rezolvate cu aceasta ocazie:

  • credentialele volunt-app sunt email/password. Dar userii au mai multe email-uri in profil si este contraintuitiv ca sa foloseasca parole diferite pentru fiecare email folosit. Ar trebuii ca parola sa fie asociata cu un profil, nu cu un email, si sa accepte login cu oricare email valid si parola corecta. Este de dorit ca migrarea la credentiale profil/password sa nu forteze userii sa re-introduca parole.

  • stocare email in profil ca JSON nu este optima, permite duplicate, nu este uniqueness la DB level

  • Daca se editeaza email-ul userii au probleme sa-si regaseasca profilul #27 schimbarea de email prinicpal la profil este problematica

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

1 participant