Skip to content

[PasswordHasher][Security] Remove legacy password encoders #41960

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

Merged
merged 1 commit into from
Jul 4, 2021

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Jul 3, 2021

Q A
Branch? 6.0
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Time to clean :)

@chalasr chalasr requested a review from wouterj as a code owner July 3, 2021 15:19
@carsonbot carsonbot added this to the 6.0 milestone Jul 3, 2021
@carsonbot carsonbot changed the title [Security][PasswordHasher] Remove legacy password encoders [PasswordHasher][Security] Remove legacy password encoders Jul 3, 2021
@chalasr chalasr force-pushed the remove-encoders branch from 3d344fe to c22d5f0 Compare July 3, 2021 15:22
@chalasr
Copy link
Member Author

chalasr commented Jul 3, 2021

Builds failures unrelated/false positives. Ready

@chalasr chalasr force-pushed the remove-encoders branch from c22d5f0 to 2dea43c Compare July 3, 2021 15:51
@chalasr chalasr force-pushed the remove-encoders branch from 2dea43c to fa37025 Compare July 4, 2021 10:05
@chalasr
Copy link
Member Author

chalasr commented Jul 4, 2021

Rebased.

@fabpot
Copy link
Member

fabpot commented Jul 4, 2021

Thank you @chalasr.

@fabpot fabpot merged commit d84e37a into symfony:6.0 Jul 4, 2021
@chalasr chalasr deleted the remove-encoders branch July 4, 2021 12:20
chalasr added a commit that referenced this pull request Nov 3, 2021
…cyweb)

This PR was merged into the 6.0 branch.

Discussion
----------

[SecurityBundle] Remove dead "encoder" complex type

| Q             | A
| ------------- | ---
| Branch?       | 6.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

It looks like it's a leftover from #41960

Commits
-------

dd905b2 [SecurityBundle] Remove dead "encoder" complex type
@fabpot fabpot mentioned this pull request Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants