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

contrib/google.golang.org/api: update grpc contrib to work with API changes. #566

Closed
wants to merge 1 commit into from

Conversation

knusbaum
Copy link
Contributor

@knusbaum knusbaum commented Jan 13, 2020

Google's grpc API has changed, and so our contrib no longer compiles.
This commit updates the contrib package to use the new API.

@knusbaum knusbaum added this to the 1.21.0 milestone Jan 13, 2020
…metadata API changes.

Google's metadata API has changed, and so our grpc.v12 contrib no longer compiles.
This commit updates the grpc.v12 contrib package to use the new API.
@knusbaum knusbaum changed the title contrib/google.golang.org/grpc.v12: update grpc contrib to work with metadata API changes. contrib/google.golang.org/api: update grpc contrib to work with API changes. Jan 13, 2020
@knusbaum
Copy link
Contributor Author

I had a misunderstanding about what the issue was. Working now on resolving it.

@knusbaum
Copy link
Contributor Author

This is not something we can fix, but it looks like grpc is fixing it now.

grpc deleted an API in grpc/grpc-go#3301
That API is used by google.golang.org/api: https://github.com/googleapis/google-api-go-client/blob/master/option/option.go#L153-L154

The change caused google.golang.org to stop compiling.
It looks like they are reverting the change here: grpc/grpc-go#3315

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

Successfully merging this pull request may close these issues.

None yet

1 participant