Skip to content

Comments

Update password4j docs to use BcryptPassword4jPasswordEncoder#18232

Merged
rwinch merged 1 commit intospring-projects:mainfrom
nidhogg5:main
Feb 19, 2026
Merged

Update password4j docs to use BcryptPassword4jPasswordEncoder#18232
rwinch merged 1 commit intospring-projects:mainfrom
nidhogg5:main

Conversation

@nidhogg5
Copy link
Contributor

Fix: Corrected class name inconsistency in password-storage.adoc example

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 28, 2025
@rwinch rwinch changed the title Update BcryptPassword4jPasswordEncoder example in password-storage.adoc Update password4j docs to use BcryptPassword4jPasswordEncoder Dec 1, 2025
@rwinch rwinch self-assigned this Dec 1, 2025
@rwinch rwinch added in: docs An issue in Documentation or samples type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Dec 1, 2025
@rwinch
Copy link
Member

rwinch commented Dec 1, 2025

@nidhogg5 Thanks for the Pull Request! Can you please update the commit to include the Signed-Off-by trailer to all of your commit message to fix the DCO checks? You can read more about the move to using DCO on the Spring Blog.

@rwinch rwinch added the status: waiting-for-feedback We need additional information before we can continue label Dec 1, 2025
Copy link
Member

@rwinch rwinch left a comment

Choose a reason for hiding this comment

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

Replaces the usage of BCryptPasswordEncoder with
BcryptPassword4jPasswordEncoder in documentation samples.

Signed-off-by: Chen He <nidhogg55555@gmail.com>
@rwinch rwinch closed this Feb 19, 2026
@rwinch rwinch reopened this Feb 19, 2026
@rwinch rwinch removed the status: waiting-for-feedback We need additional information before we can continue label Feb 19, 2026
@rwinch rwinch added this to the 7.1.0-M3 milestone Feb 19, 2026
@rwinch rwinch enabled auto-merge (rebase) February 19, 2026 20:19
@rwinch rwinch merged commit d65625e into spring-projects:main Feb 19, 2026
7 checks passed
@rwinch
Copy link
Member

rwinch commented Feb 19, 2026

Thanks for the Pull Request! This is now merged into main 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: docs An issue in Documentation or samples type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants