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

[make:security:custom] create a custom authenticator #1522

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

jrushlow
Copy link
Collaborator

@jrushlow jrushlow commented Apr 21, 2024

Replaces the existing make:auth command that was originally used to implement authentication with "Guard" based authentication. When we created make:security:form-login - our aim was to ultimately remove make:auth.

  • deprecates make:auth and it's MakeAuthenticator::class - deprecated in [make:auth] deprecate command #1523 for better changelog / release visibility
  • introduces make:security:custom
  • the "custom authenticator" generated replicates the example provided in the security docs: https://symfony.com/doc/current/security/custom_authenticator.html
  • make:security:custom is intended to be no-frills... It will not work out of the box because it's a "custom" authenticator. The custom implementation details should be generated in userland.

@jrushlow jrushlow added the Feature New Feature label Apr 21, 2024
@jrushlow jrushlow force-pushed the feature/security-custom branch 2 times, most recently from b4ed9e7 to 17610a5 Compare April 21, 2024 07:39
@jrushlow jrushlow changed the title WIP - [make:security:custom] create a custom authenticator [make:security:custom] create a custom authenticator Apr 21, 2024
@jrushlow jrushlow marked this pull request as ready for review April 21, 2024 07:43
@jrushlow jrushlow merged commit ed3465b into symfony:main Apr 24, 2024
7 checks passed
@jrushlow jrushlow deleted the feature/security-custom branch April 24, 2024 09:01
@jrushlow jrushlow mentioned this pull request Apr 24, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant