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

feat: add Token RefreshAPI to data plane #1120

Conversation

paullatzelsperger
Copy link
Contributor

@paullatzelsperger paullatzelsperger commented Mar 11, 2024

WHAT

This PR adds the Refresh API to the dataplane, effectively exposing the possibility to refresh dataplane access tokens.

The Refresh API is registered into th "public" context, thus it is available under POST <HOST>:<PUBLIC_API_PORT>/<PUBLIC_API_PATH>/v1/token.
Using all defaults, that is:

POST http://localhost:8185/api/v1/public/token

or - using the v2 endpoint:

POST http://localhost:8185/api/v2/public/token

using the V2 endpoint is highly encouraged as the V1 endpoint will not use data plane signaling, access tokens or token refresh!

WHY

Short-lived tokens are a security best-practice

FURTHER NOTES

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Closes # <-- insert Issue number if one exists

@paullatzelsperger paullatzelsperger added Feature critical All issues that will definitely be in the milestone they are planned for labels Mar 11, 2024
@paullatzelsperger paullatzelsperger force-pushed the feat/1109_dataplane_token_refresh_api branch 2 times, most recently from 9116d0f to 54fff11 Compare March 11, 2024 14:40
@paullatzelsperger paullatzelsperger changed the title feat: add Token RefreshAPI to data plane" feat: add Token RefreshAPI to data plane Mar 11, 2024
@paullatzelsperger paullatzelsperger force-pushed the feat/1109_dataplane_token_refresh_api branch from cbd530b to 1c5850f Compare March 12, 2024 07:49
@paullatzelsperger paullatzelsperger force-pushed the feat/1109_dataplane_token_refresh_api branch 2 times, most recently from 71408db to 0bde1e3 Compare March 12, 2024 14:56
@paullatzelsperger paullatzelsperger force-pushed the feat/1109_dataplane_token_refresh_api branch from 0bde1e3 to 8f0a249 Compare March 12, 2024 15:00
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.0% Duplication on New Code

See analysis details on SonarCloud

@paullatzelsperger paullatzelsperger merged commit 3c9e166 into eclipse-tractusx:main Mar 12, 2024
25 checks passed
@paullatzelsperger paullatzelsperger deleted the feat/1109_dataplane_token_refresh_api branch March 12, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical All issues that will definitely be in the milestone they are planned for Feature
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants