J’ai déjà remarqué ce bug sur Meta, mais il est apparu sur mes déploiements après la dernière mise à niveau : en cliquant sur connexion, la fenêtre modale « Bienvenue de retour » apparaît, et est immédiatement masquée par « Désolé, une erreur s’est produite ». Cliquer sur OK supprime la fenêtre modale d’erreur et vous pouvez reprendre l’authentification normalement.
Il y a un journal d’erreurs dans la console :
ajax-error.js:9:12
l ajax-error.js:9
p ajax-error.js:95
passkeyLogin login.js:147
2 « J'aime »
Moin
Février 1, 2024, 12:38
2
Quel navigateur utilisez-vous ?
Firefox 115.7.0esr (64 bits) sur Debian 12.
Discourse v3.3.0.beta1-dev — Commits · discourse/discourse · GitHub — Ember v5.5.0
1 « J'aime »
Moin
Février 1, 2024, 12:41
4
2 « J'aime »
Ceci est une régression introduite ici : FEATURE: Allow users to confirm session with passkeys (#24337) · discourse/discourse@c6ead3f · GitHub
) {
if (!isWebauthnSupported()) {
return errorCallback(I18n.t("login.security_key_support_missing_error"));
}
// we need to check isConditionalMediationAvailable for Firefox
// without it, Firefox will throw console errors
// We cannot do a general check because iOS Safari and Chrome in Selenium quietly support the feature
// but they do not support the PublicKeyCredential.isConditionalMediationAvailable() method
if (mediation === "conditional" && isFirefox) {
const isCMA = await PublicKeyCredential.isConditionalMediationAvailable();
if (!isCMA) {
return;
}
}
try {
const resp = await ajax("/session/passkey/challenge.json");
const credential = await navigator.credentials.get({
publicKey: {
Je pense que le code doit être déplacé dans le try{} ci-dessous ; sinon, l’erreur utilisera l’interface utilisateur au lieu de la console.
userHandle: bufferToBase64(credential.response.userHandle),
};
} catch (error) {
if (error.name === "AbortError") {
// no need to show an error when the cancelling a pending ceremony
// this happens when switching from the conditional method (username input autofill)
// to the optional method (login button) or vice versa
return null;
}
if (mediation === "conditional") {
// The conditional method gets triggered in the background
// it's not helpful to show errors for it in the UI
// eslint-disable-next-line no-console
console.error(error);
return null;
}
if (error.name === "NotAllowedError") {
return errorCallback(I18n.t("login.security_key_not_allowed_error"));
} else if (error.name === "SecurityError") {
4 « J'aime »
pmusaraj
(Penar Musaraj)
Février 2, 2024, 9:40
7
Merci pour le rapport. Cela devrait être corrigé via FIX: Handle old Firefox versions that do not support isConditionalMed… by pmusaraj · Pull Request #25549 · discourse/discourse · GitHub , je n’ai pu reproduire qu’avec cette version de Firefox sur Windows/macOS, mais la correction devrait également s’appliquer aux autres systèmes d’exploitation.
3 « J'aime »
pmusaraj
(Penar Musaraj)
A fermé ce sujet ()
Février 6, 2024, 10:41
9
Ce sujet a été automatiquement fermé après 4 jours. Les nouvelles réponses ne sont plus autorisées.