Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

feat!: Update google-auth-library to v10#1702

Merged
d-goog merged 11 commits intomainfrom
update-google-auth-library
Feb 27, 2025
Merged

feat!: Update google-auth-library to v10#1702
d-goog merged 11 commits intomainfrom
update-google-auth-library

Conversation

@d-goog
Copy link
Contributor

@d-goog d-goog commented Feb 19, 2025

Description

Updating google-auth-library to v10 along with improving type-compatibility.

Impact

All of the features in google-auth-library v10 and more open AuthClient and GoogleAuth types.

Testing

Updated and streamlined tests.

Additional Information

This functionality can be improved with this feature (to remove GoogleAuth<AuthClient> references):

🦕

@d-goog d-goog requested a review from a team February 19, 2025 22:30
@d-goog d-goog requested a review from a team as a code owner February 19, 2025 22:30
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Feb 19, 2025
gax/package.json Outdated
"node-fetch": "^2.7.0",
"object-hash": "^3.0.0",
"proto3-json-serializer": "^2.0.2",
"protobufjs": "^7.3.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was an auto-repair from npm

| GrpcClientOptions
| {
/**
* @deprecated - use `authClient` for `AuthClient`s instead
Copy link
Contributor Author

Choose a reason for hiding this comment

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

GrpcClientOptions currently includes this parameter, and with the correct name; authClient.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We should create a follow-up to clean this up in the next major to avoid confusion

Copy link
Contributor

@sofisl sofisl left a comment

Choose a reason for hiding this comment

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

LGTM once tests are passing.

@d-goog
Copy link
Contributor Author

d-goog commented Feb 24, 2025

Status update: fixing the webpack config

@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 Feb 25, 2025
@d-goog
Copy link
Contributor Author

d-goog commented Feb 26, 2025

Status update: figuring out a config issue with webpack 5…

},
{
test: /node_modules[\\/]google-auth-library/,
test: /node_modules[\\/]gcp-metadata/,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignoring gcp-metadata in this context, as it is unused, while keeping auth

@d-goog d-goog added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 27, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 27, 2025
@d-goog d-goog mentioned this pull request Feb 27, 2025
@d-goog d-goog merged commit 43a8cb8 into main Feb 27, 2025
27 checks passed
@d-goog d-goog deleted the update-google-auth-library branch February 27, 2025 19:25
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.

3 participants