Skip to content
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

Libraries BOM: gRPC and google-auth-library version upgrade #1865

Merged
merged 2 commits into from Dec 8, 2020

Conversation

suztomo
Copy link
Contributor

@suztomo suztomo commented Dec 7, 2020

update dependency com.google.auth:google-auth-library-bom to v0.22.1
update io.grpc.version to v1.34.0

The build succeeded in my local without showing the error in #1779. Does this work?

update dependency com.google.auth:google-auth-library-bom to v0.22.1
update io.grpc.version to v1.34.0
@google-cla google-cla bot added the cla: yes label Dec 7, 2020
@suztomo suztomo changed the title gRPC and google-auth-library version upgrade Libraries BOM: gRPC and google-auth-library version upgrade Dec 7, 2020
@suztomo suztomo requested a review from elharo December 7, 2020 21:09
@suztomo
Copy link
Contributor Author

suztomo commented Dec 7, 2020

Somehow kokoro-macos does not trigger the build. Posted a note to kokoro-users@.

@suztomo suztomo merged commit 27c0aaa into master Dec 8, 2020
@elharo
Copy link
Contributor

elharo commented Dec 8, 2020

Per @ejona "It may be best to revert this change, or hold off on releasing until you can bump to a 1.34.1. I'm currently investigating grpc/grpc-java#7643, but it appears 1.34.0 has bad credential handling that would impact direct path for some Google properties."

I don't think we need to revert but let's not release until this is resolved.

@elharo elharo deleted the google-auth-library_grpc branch December 8, 2020 12:01
@suztomo
Copy link
Contributor Author

suztomo commented Dec 8, 2020

Good information. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants