Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Security] Integrated Guards with the Authenticator system#36570

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fabpot merged 1 commit intosymfony:masterfromwouterj:security/new-system-guard
Apr 25, 2020

Conversation

wouterj
Copy link
Member

@wouterjwouterj commentedApr 24, 2020•
edited
Loading

QA
Branch?master
Bug fix?no
New feature?yes
Deprecations?no
Tickets-
LicenseMIT
Doc PR-

This way, the guard configuration (and guard authenticators) can use the new authenticator system as well. Advantages:

  • Any bundle providing guard integration (e.g. Lexik JWT, Knp Oauth) can now integrate with the authenticator system as well
  • (after we've integrated LDAP as well) Anyone can setsecurity.enable_authenticator_manager: true to test the new experimental system without needing to update any PHP code.

cc@weaverryan

sstok reacted with hooray emoji
@wouterjwouterj changed the titleIntegrated Guards with the Authenticator system[Security] Integrated Guards with the Authenticator systemApr 24, 2020
@wouterjwouterjforce-pushed thesecurity/new-system-guard branch fromb0b5136 to985c70cCompareApril 24, 2020 16:49
@wouterjwouterjforce-pushed thesecurity/new-system-guard branch from985c70c to8708a6cCompareApril 24, 2020 17:08
Copy link
Member

@chalasrchalasr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

👍 Worth a CHANGELOG entry?

@wouterj
Copy link
MemberAuthor

wouterj commentedApr 25, 2020•
edited
Loading

Worth a CHANGELOG entry?

We only have a CHANGELOG file insymfony/security and we didn't document any other new authenticator there as well (only "hey, we wrote a complete new system"). Do you want me to add an entry with all new authenticators?

@chalasr
Copy link
Member

Ok thanks, nothing to do then :)
Ready to go on my side.

@fabpot
Copy link
Member

Thank you@wouterj.

@fabpotfabpot merged commit6b48511 intosymfony:masterApr 25, 2020
@wouterjwouterj deleted the security/new-system-guard branchApril 25, 2020 14:39
@nicolas-grekasnicolas-grekas modified the milestones:next,5.1May 4, 2020
@fabpotfabpot mentioned this pull requestMay 5, 2020
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@fabpotfabpotfabpot approved these changes

@chalasrchalasrchalasr approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.

5 participants
@wouterj@chalasr@fabpot@nicolas-grekas@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp