Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

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

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

aeitzman merged 12 commits into
googleapis: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

sonarqubecloud Bot commented Feb 2, 2024

Copy link
Copy Markdown

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?

@BigTailWolf BigTailWolf left a comment

Copy link
Copy Markdown
Contributor

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants