Skip to content
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

[Stimulus/Symfony UX] Allow a remember-me checkbox at login (or document it, if it already works somehow) #673

Open
spackmat opened this issue Jan 9, 2025 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@spackmat
Copy link

spackmat commented Jan 9, 2025

Description

Hi, one login form of mine has a checkbox for the remember-me feature of Symfony. I implemented an optional passkey-login for that form, which works great with one caveat: The checkbox for remember-me is ignored and thus users logging in with a Passkey are not remembered and must login every time they come back or their session is expired. I could enable always remember me in Symfony, but for that project that feature shall be optional.

I did not find a documented way to get the Symfony UX controller to work with a remember-me checkbox.

Another very similar feature, I could not find, would be to allow a form field for an authenticator name (and pass that to my authenticator entity) when a new authenticator is registered.

Example

No response

@spackmat
Copy link
Author

In think both features would be nice to have, but for now, we decided to ditch the optional remember-me and made id mandatory. And for the name of the authenticators, we implemented as a workaround a name on our WebauthnCredential entity and a little AJAX-Form to change it later, so our users can use their own names instead of the Uuids of their Passkeys.

Additional to that, we implemented a Symfony service that keeps the a local copy of the list of provider names from https://github.com/passkeydeveloper/passkey-authenticator-aaguids updated and resolves the aaguids of the passkeys to user readable names. That name-resolving would also be a nice feature for the bundle by the way.

@Spomky
Copy link
Contributor

Spomky commented Jan 15, 2025

Hello @spackmat,

I link this issue with #460.
Indeed, You should be allowed to define your RememberMe badges.

@Spomky Spomky self-assigned this Jan 15, 2025
@Spomky Spomky added the enhancement New feature or request label Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants