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

Docs: security-jdbc guide linking user entity section where is used built-in bcrypt password mapper #37771

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

jcarranzan
Copy link
Contributor

As I describe in point 1 from #37682 :
In the "Configuring the Application" section, from https://quarkus.io/version/main/guides/security-jdbc, there is a clear-text password used in the INSERTS command SQL script. A note reminds us not to store clear-text passwords in production environments. Additionally, it is mentioned that the elytron-security-jdbc offers a built-in bcrypt password mapper.
However, it would be great to show how to implement this or link to another source demonstrating the process.

Copy link

github-actions bot commented Dec 15, 2023

🙈 The PR is closed and the preview is expired.

@quarkus-bot quarkus-bot bot added the area/docstyle issues related for manual docstyle review label Dec 15, 2023
…cryptUtil; is used demonstrating the password mapper process

Call an anchored ID from a different file
@jcarranzan jcarranzan force-pushed the tweaks-security-jdbc-adoc branch from 2831196 to 73c6bbb Compare December 15, 2023 12:58
Copy link
Member

@sberyozkin sberyozkin left a comment

Choose a reason for hiding this comment

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

@sberyozkin sberyozkin merged commit 0b2706f into quarkusio:main Dec 15, 2023
5 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.7 - main milestone Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docstyle issues related for manual docstyle review area/documentation
Projects
Development

Successfully merging this pull request may close these issues.

3 participants