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
feat: Support instantiating GoogleAuth
with an AuthClient
#1364
Conversation
This will allow downstream clients to support features like Downscoped Tokens without breaking changes
Warning: This pull request is touching the following templated files:
|
This test has silently failed for some time as the old bucket did not exist. Fixing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This implementation seems super clean, great work.
I'm always somewhat cautious with updates to the client library, perhaps you could link and run the storage sample/system tests against this branch? at which point let's ship.
return opts; | ||
}, | ||
}, | ||
authClient: new GoogleAuth({auth: oauth2Client}), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, this is much nicer.
// GCS object name. | ||
const objectName = 'cab-first-z2zsauf4sj.txt'; | ||
const objectName = 'cab-first-"brd3qlsuok.txt'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curious why we changed this object name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question: The bucket and object previously didn’t exist - and the test has silently failed until this PR as we were asserting a string that existed in the error message (‘first’).
This will allow downstream clients to support features like Downscoped Tokens without breaking changes