-
Notifications
You must be signed in to change notification settings - Fork 2k
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
googlesapis 49 causes failing tests in storage transfer #1747
Comments
I feel like this is actually @laylasells's domain. |
@laylasells Could somebody look into this please? We're now 3 releases behind because we can't update any sample in the repo. Thanks! |
Hi @fhinkel, I'm taking a look. Apologies for the delay. |
The example is inconsistent with how auth is currently documented at https://cloud.google.com/storage-transfer/docs/reference/rest/v1/transferJobs/list. I saw consistent failures of unauthorized failures. Then I found: https://github.com/googleapis/google-auth-library-nodejs/blob/41b0cb7b3dbae1b264b7de1cae82648c402d5cc3/CHANGELOG.md#v200 I was able to fix the broken tests locally after updating the auth initialization in #1807. I want to make sure CI passes, but this might be the issue given auth is at 5.0.0 IIUC. |
See #1784
The text was updated successfully, but these errors were encountered: