Skip to content

feat: Add context object to pass to supplier functions#1363

Merged
aeitzman merged 12 commits intogoogleapis:mainfrom
aeitzman:add-context
Feb 2, 2024
Merged

feat: Add context object to pass to supplier functions#1363
aeitzman merged 12 commits intogoogleapis:mainfrom
aeitzman:add-context

Conversation

@aeitzman
Copy link
Copy Markdown
Contributor

Adds context object to pass from credentials to user defined suppliers.

@aeitzman aeitzman requested review from a team January 31, 2024 00:06
@product-auto-label product-auto-label Bot added the size: m Pull request size is medium. label Jan 31, 2024
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Jan 31, 2024
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
Comment thread oauth2_http/java/com/google/auth/oauth2/ExternalAccountSupplierContext.java Outdated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 2, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@aeitzman aeitzman merged commit 1d9efc7 into googleapis:main Feb 2, 2024
@@ -0,0 +1,100 @@
package com.google.auth.oauth2;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Add a copyright header?

Copy link
Copy Markdown
Contributor

@BigTailWolf BigTailWolf left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants