-
Notifications
You must be signed in to change notification settings - Fork 212
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
token-auth: add vault support #1545
token-auth: add vault support #1545
Conversation
...src/main/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtension.java
Outdated
Show resolved
Hide resolved
Ah sorry, didn't saw, that the PR is still a draft 🙈 |
Feedback is always welcome :) |
ff8897f
to
6135138
Compare
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
6135138
to
0895729
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of minor nits apart from the dependency injection stuff
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## main #1545 +/- ##
==========================================
+ Coverage 67.53% 67.57% +0.04%
==========================================
Files 737 737
Lines 16201 16206 +5
Branches 1058 1060 +2
==========================================
+ Hits 10941 10952 +11
+ Misses 4782 4776 -6
Partials 478 478
Continue to review full report at Codecov.
|
f846b82
to
a0d6d42
Compare
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
...test/java/org/eclipse/dataspaceconnector/api/auth/TokenBasedAuthenticationExtensionTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Dominik Pinsel <dominik.pinsel@daimler.com> Co-authored-by: Paul Latzelsperger <43503240+paullatzelsperger@users.noreply.github.com>
766fa9b
to
4fd308c
Compare
Signed-off-by: Dominik Pinsel <dominik.pinsel@daimler.com> Co-authored-by: Paul Latzelsperger <43503240+paullatzelsperger@users.noreply.github.com> Co-authored-by: Paul Latzelsperger <43503240+paullatzelsperger@users.noreply.github.com>
What this PR changes/adds
Extension for token based authentication can now retrieve the authentication key from a vault.
see issue #1537
Why it does that
Further notes
Linked Issue(s)
Closes #1537
Checklist
no-changelog
)Dominik Pinsel dominik.pinsel@daimler.com, Mercedes-Benz Tech Innovation GmbH, legal info/Impressum