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

Upgrade google-http-client and google-http-client-gson. #16082

Closed
wants to merge 1 commit into from

Conversation

tjgq
Copy link
Contributor

@tjgq tjgq commented Aug 10, 2022

In #15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes #15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4

@tjgq tjgq marked this pull request as draft August 10, 2022 15:56
@tjgq tjgq force-pushed the auth branch 2 times, most recently from a416cea to a8fbbdb Compare August 26, 2022 10:14
@tjgq tjgq requested a review from meteorcloudy August 26, 2022 10:24
In bazelbuild#15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes bazelbuild#15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4
@tjgq tjgq marked this pull request as ready for review August 26, 2022 10:25
@meteorcloudy
Copy link
Member

@sgowroji Can you help import this one? Thanks!

@sgowroji sgowroji added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally team-Remote-Exec Issues and PRs for the Execution (Remote) team labels Aug 26, 2022
copybara-service bot pushed a commit that referenced this pull request Aug 27, 2022
In #15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes #15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4

Partial commit for third_party/*, see #16082.

Signed-off-by: Sunil Gowroji <sgowroji@google.com>
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Aug 27, 2022
@sgowroji
Copy link
Member

Merged at 0696b8a

@sgowroji sgowroji closed this Aug 27, 2022
aiuto pushed a commit to aiuto/bazel that referenced this pull request Oct 12, 2022
In bazelbuild#15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes bazelbuild#15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4

Partial commit for third_party/*, see bazelbuild#16082.

Signed-off-by: Sunil Gowroji <sgowroji@google.com>
tjgq added a commit to tjgq/bazel that referenced this pull request Nov 11, 2022
In bazelbuild#15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes bazelbuild#15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4

Partial commit for third_party/*, see bazelbuild#16082.

Signed-off-by: Sunil Gowroji <sgowroji@google.com>
ShreeM01 pushed a commit that referenced this pull request Nov 15, 2022
In #15176 we upgraded google-auth-library-oauth2-http to 1.6.0, but didn't
upgrade its dependencies accordingly; Maven claims 1.41.4 is needed [1].
In turn, a new transitive dependency on opencensus-contrib-http-util 0.31.0
also becomes necessary [2].

Fixes #15639.

[1] https://mvnrepository.com/artifact/com.google.auth/google-auth-library-oauth2-http/1.6.0
[2] https://mvnrepository.com/artifact/com.google.http-client/google-http-client/1.41.4

Partial commit for third_party/*, see #16082.

Signed-off-by: Sunil Gowroji <sgowroji@google.com>

Signed-off-by: Sunil Gowroji <sgowroji@google.com>
@tjgq tjgq deleted the auth branch December 8, 2022 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bazel 5.2 Google Cloud's Workload identity federation auth seems broken
3 participants