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

Misspelling in generated JwtAuthenticationTestUtils.java #25079

Closed
1 task done
timothystone-knsl opened this issue Feb 4, 2024 · 2 comments · Fixed by #25080
Closed
1 task done

Misspelling in generated JwtAuthenticationTestUtils.java #25079

timothystone-knsl opened this issue Feb 4, 2024 · 2 comments · Fixed by #25080

Comments

@timothystone-knsl
Copy link
Contributor

timothystone-knsl commented Feb 4, 2024

Overview of the issue

Observed a misspelling in the JwtAuthenticationTestUtils.java.ejs template using the JwtClaimsSet.builder setting claims via a Lambda expression.

Motivation for or Use Case

Tidiness.

Reproduce the error

Observe diff. See generators/server/templates/src/test/java/_package_/security/jwt/JwtAuthenticationTestUtils.java.ejs.

Related issues

None.

Suggest a Fix

Correct misspelling. See PR #25080

JHipster Version(s)

8.1.0

JHipster configuration

Not applicable.

Browsers and Operating System

Not applicable.

  • Checking this box is mandatory (this is just to show you read everything)
timothystone-knsl added a commit to timothystone-knsl/generator-jhipster that referenced this issue Feb 4, 2024
Observed a misspelling in the JwtAuthenticationTestUtils.java.ejs template using the
JwtClaimsSet.builder setting claims via a Lambda expression.

s/customClain/customClaim

Fixes jhipster#25079
mshima pushed a commit that referenced this issue Feb 4, 2024
Observed a misspelling in the JwtAuthenticationTestUtils.java.ejs template using the
JwtClaimsSet.builder setting claims via a Lambda expression.

s/customClain/customClaim

Fixes #25079
@timothystone-knsl
Copy link
Contributor Author

🙏 Thanks @mshima.

@mshima
Copy link
Member

mshima commented Feb 4, 2024

@timothystone-knsl thank you for the contribution.

@deepu105 deepu105 added this to the 8.2.0 milestone Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants