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

Guide on mTLS should include an additional dependency #1727

Open
azurvii opened this issue Jun 15, 2023 · 0 comments
Open

Guide on mTLS should include an additional dependency #1727

azurvii opened this issue Jun 15, 2023 · 0 comments

Comments

@azurvii
Copy link

azurvii commented Jun 15, 2023

Describe the bug

SecurityIdentity seems to require the quarkus-security dependency, for mTLS at least. Without it, explicitly or implicitly, the injection would fail with UnsatisfiedResolutionException. I got around the error by adding the dep, following the suggestion in quarkusio/quarkus#25122 (comment).

I suggest to mention the need for the dependency (I don't know if quarkus-security is the lowest-level key dependency; but looking at its transitive dependencies, this looks most appropriate). This could save hours for other new comers, like me.

An alternative, a better one if feasible, is to include the common cause in the UnsatisfiedResolutionException thrown, stating that it's possibly missing quarkus-security (or something else that is actionable).

Which pages

https://quarkus.io/guides/security-authentication-mechanisms-concept#mutual-tls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant