Dall’inizio, i social login su Discourse da desktop utilizzavano sempre una finestra “popup” per l’accesso.
L’intento era essere “meno” distraenti, mantenendoti sul sito effettivo mentre avveniva l’azione di “accesso”.
Penso che non abbia più senso mantenere questa modalità per diversi motivi.
È già incoerente con il modo in cui funziona il “SSO di Discourse”, può essere instabile sui browser che non consentono i popup (il mio Firefox si lamenta e richiede un clic per abilitarlo), il codice per pubblicare messaggi è complesso e difficile da mantenere, inoltre ora richiediamo già un clic aggiuntivo sul popup.
Come vi sentireste a eliminare il codice relativo all’apertura di una finestra “popup” per i social login e ad avere un unico metodo coerente per l’accesso, sia che si tratti di SSO, social login o altro?
Una volta che verrà finalmente inclusa nella versione stabile, dovremo rimuovere le opzioni di configurazione ridondanti “schermo intero” dai plugin (ad esempio OAuth2, OIDC)
Stiamo modificando alcune parti piuttosto critiche del nostro codice di autenticazione, quindi il ramo è ora distribuito su Meta per i test.
Se qualcuno ha un po’ di tempo a disposizione, provate a effettuare l’accesso utilizzando diversi provider e a disconnettere/rimettere in connessione gli account nelle vostre preferenze. Se riscontrate problemi, fatemelo sapere e procederò all’indagine
È anche un’ottima occasione per provare la nostra nuova autenticazione Discord, configurata qui su Meta
Ho testato l’accesso con Facebook, Google, Twitter e Discord: tutto andato a buon fine. Ho inoltre testato la disconnessione e la riconnessione degli stessi account dalle preferenze dell’utente. Entrambe le procedure sono state fluide.
Non posso testare GitHub poiché ho l’autenticazione a due fattori (2FA) attiva sull’account Discourse collegato al mio GitHub.
Ho eseguito ulteriori test (i test precedenti erano stati effettuati tutti su Chrome in Windows). Ho confermato che l’accesso funziona anche con IE, Edge e Firefox su Windows. Ho individuato due piccoli bug, uno relativo alla disconnessione e l’altro alla connessione dell’account in IE, ma è stato confermato che entrambi esistevano già prima di questa modifica.
Si trattava di un problema di cache che interessava solo IE11. Dovrebbe essere risolto da questo commit. Stiamo distribuendo questa correzione su Meta ora
