Skip to content

Conversation

larsgrefer
Copy link
Contributor

see #7042

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 28, 2019
@rwinch rwinch self-assigned this Jul 3, 2019
@rwinch rwinch closed this in e95effc Jul 3, 2019
rwinch added a commit that referenced this pull request Jul 3, 2019
* Break up tests
* Rename test methods to follow conventions
* Fix checkstyle

Issue gh-7057
@rwinch rwinch added in: crypto An issue in spring-security-crypto type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 3, 2019
@rwinch rwinch added this to the 5.2.0.RC1 milestone Jul 3, 2019
@rwinch
Copy link
Member

rwinch commented Jul 3, 2019

Thanks for the PR @larsgrefer! This is now merged into master with a bit of polish e1f155b

rwinch added a commit that referenced this pull request Jul 3, 2019
* Break up tests
* Rename test methods to follow conventions
* Fix checkstyle

Fixes gh-7057

# Conflicts:
#	crypto/src/main/java/org/springframework/security/crypto/scrypt/SCryptPasswordEncoder.java
#	crypto/src/test/java/org/springframework/security/crypto/scrypt/SCryptPasswordEncoderTests.java
@spring-projects-issues spring-projects-issues added the status: backported An issue that has been backported to maintenance branches label Jul 3, 2019
@rwinch rwinch removed the status: backported An issue that has been backported to maintenance branches label Jul 3, 2019
kostya05983 pushed a commit to kostya05983/spring-security that referenced this pull request Aug 26, 2019
kostya05983 pushed a commit to kostya05983/spring-security that referenced this pull request Aug 26, 2019
* Break up tests
* Rename test methods to follow conventions
* Fix checkstyle

Issue spring-projectsgh-7057
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: crypto An issue in spring-security-crypto type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants