Skip to content
This repository has been archived by the owner on May 9, 2022. It is now read-only.

Consentire avvio in modalità headless (con login automatico). Fix issue #56 #118

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lussoluca
Copy link
Member

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.

@lussoluca lussoluca changed the title #56 - Consentire avvio in modalità headless (con login automatico) Consentire avvio in modalità headless (con login automatico). Fix issue #56 Aug 9, 2018
@alranel alranel changed the title Consentire avvio in modalità headless (con login automatico). Fix issue #56 WIP Consentire avvio in modalità headless (con login automatico). Fix issue #56 Aug 9, 2018
@alranel
Copy link
Member

alranel commented Aug 9, 2018

Perché anche la password nel parametro auto_login? Non basta l'username?

@alranel
Copy link
Member

alranel commented Aug 9, 2018

In caso di login fallito dovresti ritornare la Response con AuthnFailed, come da regole SPID/SAML.

@alranel
Copy link
Member

alranel commented Aug 9, 2018

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?

@lussoluca
Copy link
Member Author

@alexrj ho rimosso la password da auto_login e ora ritorno AuthnFailed se lo username passato non esiste

@alranel
Copy link
Member

alranel commented Aug 17, 2018

Grazie @lussoluca!
Lascio a @fmarco e @sanjioh la review e il merge.

@fmarco
Copy link
Member

fmarco commented Aug 21, 2018

@lussoluca @alexrj la metterei un attimo in standby, in modo da riallinearla dopo il merge dell'altra PR

@alranel alranel changed the title WIP Consentire avvio in modalità headless (con login automatico). Fix issue #56 Consentire avvio in modalità headless (con login automatico). Fix issue #56 Sep 11, 2018
@fmarco
Copy link
Member

fmarco commented Sep 18, 2018

@lussoluca Ciao, riusciresti ad allinearti con l'attuale master?

@lussoluca
Copy link
Member Author

@fmarco fatto

@lussoluca
Copy link
Member Author

ok no, ci sono un po' di test da sistemare :-(

@bfabio
Copy link
Member

bfabio commented Dec 22, 2020

@lussoluca it would be great if you could resume your work on this :)

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

Successfully merging this pull request may close these issues.

None yet

4 participants