Skip to content

docs(symfony-ux): fix wrong route name and Stimulus action (5.3) - #58

Merged
Spomky merged 1 commit into
5.3from
fix/893-stimulus-doc-route-action-5.3
May 17, 2026
Merged

Spomky merged 1 commit into
5.3from
fix/893-stimulus-doc-route-action-5.3

Conversation

@Spomky

@Spomky Spomky commented May 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

5.3 leg of the 5.2 -> 5.3 -> 5.4 cascade started in #57.

Reported in web-auth/webauthn-framework#893: the registration snippets in symfony-ux/ referenced a non-existent route and a non-existent Stimulus action, so users following the docs verbatim saw the Register button do nothing.

  • path('webauthn.controller.creation.creation.<X>') is not a real route. The bundle exposes webauthn.controller.creation.request.<X> for the options endpoint (.response.<X> for the result endpoint).
  • stimulus_action('@web-auth/webauthn-stimulus', 'register') targets a method that does not exist on the unified controller. The unified controller exposes signup and signin. The register action only exists on the split @web-auth/webauthn-stimulus/registration controller, which is unaffected here.

Files touched:

  • symfony-ux/installation.md (route fix only on 5.3, the action was already on the split controller)
  • symfony-ux/user-registration.md
  • symfony-ux/additional-authenticators.md

Two long-standing mistakes in the registration snippets caused user
reports of "the Register button does nothing":

- `path('webauthn.controller.creation.creation.<X>')` is not a real
  route. The bundle exposes `webauthn.controller.creation.request.<X>`
  for the options endpoint (and `.response.<X>` for the result endpoint).
- `stimulus_action('@web-auth/webauthn-stimulus', 'register')` targets
  a method that does not exist on the unified controller. The unified
  controller exposes `signup` and `signin`. The `register` action only
  exists on the split `@web-auth/webauthn-stimulus/registration`
  controller, which is unaffected.

Reported in web-auth/webauthn-framework#893.
@Spomky Spomky added bug Something isn't working documentation Improvements or additions to documentation labels May 17, 2026
@Spomky Spomky self-assigned this May 17, 2026
@Spomky
Spomky merged commit 100b331 into 5.3 May 17, 2026
3 checks passed
@Spomky
Spomky deleted the fix/893-stimulus-doc-route-action-5.3 branch May 17, 2026 20:28
Spomky added a commit that referenced this pull request Aug 30, 2026
* 5.3:
  docs: cover the WebAuthn Level 3 compliance fixes of 5.3.6
  docs(symfony-ux): mention the TypeScript declarations shipped since 5.3.3
  docs(repository): explain when the legacy repository alias is kept
  docs(fake-credentials): document the secret required by the simple generator
  docs(relying-party): reflect the scoped rp.name deprecation and its serialization default
  docs(conditional-create): the user verification check is relaxed too
  docs(cross-origin): document verbatim facet IDs and the mandatory top origin validator
  docs(symfony-ux): fix wrong route name and Stimulus action (#893) (#58)

# Conflicts:
#	symfony-ux/installation.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant