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

Add support for OIDC token exchange in Access manager #1103

Merged
merged 12 commits into from
Apr 6, 2025

Conversation

EyalDelarea
Copy link
Contributor

@EyalDelarea EyalDelarea commented Mar 26, 2025

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

This PR introduces support for exchanging OIDC ID tokens with access tokens in the JFrog CLI's access manager. It includes:

✅ New ExchangeOidcToken method added to AccessServicesManager

✅ Implementation of ExchangeOidcToken in TokenService, using the api/v1/oidc/token endpoint

✅ New struct CreateOidcTokenParams to represent parameters required for token exchange

✅ New struct OidcTokenResponseData to handle the response from the OIDC token exchange API

🔍 Purpose:

Enables CLI-based integration with OIDC providers by supporting the OIDC token exchange flow, as part of broader OIDC authentication support in the CLI.

@EyalDelarea EyalDelarea added the improvement Automatically generated release notes label Mar 26, 2025
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Mar 26, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 26, 2025
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Mar 27, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 27, 2025
@EyalDelarea EyalDelarea marked this pull request as ready for review March 27, 2025 09:49
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Apr 3, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Apr 3, 2025
Copy link
Contributor

github-actions bot commented Apr 6, 2025

👍 Frogbot scanned this pull request and did not find any new security issues.


@EyalDelarea EyalDelarea merged commit ee90d11 into jfrog:dev Apr 6, 2025
28 checks passed
@EyalDelarea EyalDelarea added new feature Automatically generated release notes and removed improvement Automatically generated release notes labels Apr 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants