forked from NYANDEV/forgejo
027014d7de
Follow: * #22697 There are some bugs in #22697: * https://github.com/go-gitea/gitea/pull/22697#issuecomment-1577957966 * the webauthn failure message is never shown and causes console error * The `document.getElementById('register-button')` and `document.getElementById('login-button')` is wrong * there is no such element in code * it causes JS error when a browser doesn't provide webauthn * the end user can't see the real error message These bugs are fixed in this PR. Other changes: * Use simple HTML/CSS layouts, no need to use too many `gt-` patches * Make the webauthn page have correct "page-content" layout * The "data-webauthn-error-msg" elements are only used to provide locale texts, so move them into a single "gt-hidden", then no need to repeat a lot of "gt-hidden" in code * The `{{.CsrfTokenHtml}}` is a no-op because there is no form * Many `hideElem('#webauthn-error')` in code is no-op because the `webauthn-error` already has "gt-hidden" by default * Make the tests for "URLEncodedBase64" really test with concrete cases. Screenshots: * Error message when webauthn fails (before, there is no error message): <details> ![image](https://github.com/go-gitea/gitea/assets/2114189/93cf9559-d93b-4f06-9d98-0f7032d9c65b) </details> * Error message when webauthn is unavailable <details> ![image](https://github.com/go-gitea/gitea/assets/2114189/ffc0fcd9-b93b-4418-979c-c89bb627aaf2) </details> |
||
---|---|---|
.. | ||
activate.tmpl | ||
captcha.tmpl | ||
change_passwd.tmpl | ||
change_passwd_inner.tmpl | ||
finalize_openid.tmpl | ||
forgot_passwd.tmpl | ||
grant.tmpl | ||
grant_error.tmpl | ||
link_account.tmpl | ||
oidc_wellknown.tmpl | ||
prohibit_login.tmpl | ||
reset_passwd.tmpl | ||
signin.tmpl | ||
signin_inner.tmpl | ||
signin_navbar.tmpl | ||
signin_openid.tmpl | ||
signup.tmpl | ||
signup_inner.tmpl | ||
signup_openid_connect.tmpl | ||
signup_openid_navbar.tmpl | ||
signup_openid_register.tmpl | ||
twofa.tmpl | ||
twofa_scratch.tmpl | ||
webauthn.tmpl | ||
webauthn_error.tmpl |