A runtime error preventing login via email/password occurs in Firefox (including Tor), throwing “PublicKeyCredential is not defined” during passkey-related operations. The issue affects invite-only, login-only setups and was confirmed by joerogan and Arkshine. pmusaraj identified the root cause: inconsistent browser support for isConditionalMediationAvailable(), especially in Firefox where the API is entirely unsupported. A fix was developed and merged via PR #24142pmusaraj, requiring a full rebuild to apply, as local edits to source files won’t trigger asset recompilation pmusaraj. joerogan confirmed the fix didn’t work after container restart alone. The fix is now live in tests-passed and deployed to meta.discourse.orgpmusaraj. Users are advised to rebuild their instance after the commit appears in tests-passed. Rafael_Melo_Silva reported the same error on Android, prompting pmusaraj to ask for browser version details to verify if the fix resolves mobile cases too. Temporary workaround: use one-time session links via email.
damn! thank you both and all who are fixing this. So then I run rebuild and it should be good , yeah? Or discourse-setup pulls from the repo too? Also I believe there is an error with the default onion template: the onion links redirects to the clearnet page and I have to run the doctor script becuase of some errors. I dont think it makes sense to manually turn off the redirect in nginx, since the rest of requests will still go through clearnet right?
Can I technically add those couple lines myself so I dont add more down time with rebuilding? Where is that app directory? And would launcher restart be enough?
It’s coming, it’s coming. I just pushed a modification to the fix, it’s tricky because the passkeys’ conditional UI is unevenly supported by browsers.
In Firefox. it is completely unsupported. In Safari on iOS, the feature is supported, but calling isConditionalMediationAvailable() returns false… so I had to modify the code to add a feature check just for Firefox.
Once tests pass, I will merge the change, it should be then available in about an hour.
thank yeah I saw the second commits… I am changing things locally so I wont rebuild right now and just restart container. It will work yeah? Did you just locally manually?
The fix is now merged, and I see it is deployed to meta.discourse.org, can you reproduce the issue here on your Android device? If so, please share browser version, thanks.