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

[CDAP-20957] Add scopes support to Remote Authenticator #15542

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

itsankit-google
Copy link
Member

@itsankit-google itsankit-google commented Feb 9, 2024

JIRA: CDAP-20957

context: We implemented fake metadata server in CDAP 6.10.0 which falls back to actual metadata server when namespace service account is not configured. In that case, we do not accept scopes which is an important property when fetching token for APIs which are not included in cloud platform scope like drive.googleapis.com, etc.

Before Fix:
image

After Fix:
image

@itsankit-google itsankit-google added the build Triggers github actions build label Feb 9, 2024
@itsankit-google itsankit-google changed the title [PLUGIN-1746] Add scopes support to Remote Authenticator [CDAP-20957] Add scopes support to Remote Authenticator Feb 9, 2024
@itsankit-google itsankit-google force-pushed the PLUGIN-1746 branch 2 times, most recently from 7018863 to 9e61ca3 Compare February 9, 2024 13:57
Copy link
Contributor

@dli357 dli357 left a comment

Choose a reason for hiding this comment

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

LGTM, please fix the checkstyle warnings prior to merging, thanks!

@itsankit-google itsankit-google merged commit 7192fd5 into develop Feb 13, 2024
10 of 12 checks passed
@itsankit-google itsankit-google deleted the PLUGIN-1746 branch February 13, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Triggers github actions build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants