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
Caricamento della sessione di registrazione #466
Caricamento della sessione di registrazione #466
Conversation
@PaoloGiustiniani il database di luke è poi stato resettato? |
@yakky il db di aggiorna ogni lunedì notte |
@PaoloGiustiniani è stato ripristinato? perché fino a qualche tempo fa questo non avveniva più |
@yakky sì, nelle ultime due settimane |
@PaoloGiustiniani grazie |
@yakky sto provando su luke questa cosa ma ho questo problema. Siccome luke non manda email vorrei prendere il testo del messaggio Messaggi di Posta ma il messaggio di conferma di registrazione del nuovo aspirante non viene salvato in quella tabella e di conseguenza non sono in grado di fare nessun test. A questo punto, visto che tanto gli aspiranti hanno problemi, se non hai proposte alternative, potremmo mandare tutto in produzione e testare direttamente da lì. Fammi sapere |
@luca-dex, tutte le e-mail inviate in staging vengono processate dal backend fittizio che le salva in |
Scrivevo questo perché, ad esempio, se chiedo un'estensione, trovo le email
generate per la richiesta di estensione in db.
Non ho però modo di accedere al fs di luke
…On 7 Jan 2017 14:52, "Alfio Emanuele" ***@***.***> wrote:
@luca-dex <https://github.com/luca-dex>, tutte le e-mail inviate in
staging vengono processate dal backend fittizio che le salva in
/staging/emails/. Le email sono salvate in un file il cui nome e'
praticamente il timestamp dell'invio.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#466 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADKYyYdRsQpVkP419xZ96HyNM_WPoQYjks5rP5iwgaJpZM4LYyJ_>
.
|
Ho appena visto l'issue che ha generato questa PR, visto che e' bloccante e i test funzionali di registrazione passano per me va bene mandare in produzione. |
Facciamo merge? |
@luca-dex @AlfioEmanueleFresta fatto. Avvisate quando va in produzione che eventualmente facciamo in modo di avere risorse pronte per gestire eventuali criticità |
lo tiro su produzione ora, @PaoloGiustiniani quando vuoi a questo punto aggiorna gaia |
@yakky se siete pronti aggiorno gaia |
@CroceRossaItaliana/sviluppo Ciao a tutti, Ho aggiornato Gaia, simulato una nuova registrazione e ricevuto correttamente la mail. Provando ad aprire il link da un browser diverso da quello utilizzato in fase di registrazione ottengo: Errore nel processo di registrazione. Il link di conferma che hai usato non è più valido. E' voluto? |
Ho simulato un altro scenario che ha causato l'invio multiplo di email e l'annullamento della scheda profilo:
|
Effettivamente anche io ho ricevuto la seconda email dopo aver aperto la pagina di conferma dei dati tramite il primo link ricevuto. I due link ricevuti sono differenti. |
il link ovviamente è clickabile solo una volta. Controllo l'invio della seconda email. |
@PaoloGiustiniani @luca-dex in realtà la doppia email parte quando si conferma la registrazione. Ho affinato i test e apro una PR |
@yakky ok, va bene |
Fix #454
L'id di sessione è inserito nell'url. Sono aggiunti dei check per evitare problemi di impersonation e fixation (anche se in quest'ultimo caso non può essere del tutto escluso, anche se il danno sarebbe limitato perché non sono inseriti dati particolarmente sensibili a questo stadio)
Modifica caricata su luke