-
Notifications
You must be signed in to change notification settings - Fork 38
Consentire avvio in modalità headless (con login automatico). Fix issue #56 #118
base: master
Are you sure you want to change the base?
Conversation
Perché anche la password nel parametro auto_login? Non basta l'username? |
In caso di login fallito dovresti ritornare la Response con AuthnFailed, come da regole SPID/SAML. |
Per ritornare un successo o un fallimento, più che aggiungere un parametro non basta che nel tuo script CI modifichi il parametro auto_login della configurazione impostando un utente esistente o un utente non esistente? |
@alexrj ho rimosso la password da auto_login e ora ritorno AuthnFailed se lo username passato non esiste |
Grazie @lussoluca! |
@lussoluca @alexrj la metterei un attimo in standby, in modo da riallinearla dopo il merge dell'altra PR |
@lussoluca Ciao, riusciresti ad allinearti con l'attuale master? |
@fmarco fatto |
ok no, ci sono un po' di test da sistemare :-( |
@lussoluca it would be great if you could resume your work on this :) |
Ho aggiunto un nuovo argomento
auto_login
con la combinazione di username e password da usare per fare il login automatico di un utente (username$$password
).Ho spostato un po' di codice comune in una nuova funzione
_setup_response
(che però prende un po' troppi parametri a mio avviso...).Fatemi sapere se sono sulla strada giusta.
Mancano i test, capire cosa ritornare in caso di login fallito e un ulteriore parametro per scegliere se ritornare un successo o un fallimento.