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

Bump protobuf-java #1454

Closed
wants to merge 1 commit into from

Conversation

regiskuckaertz
Copy link
Contributor

I was trying to get the latest zio grpc work with the latest scalapb but it seems like the protobuf-java version is not in sync and it is throwing a NoSuchMethod exception somewhere as the parseUnknownField has only been recently introduced (I believe due to security issues)

@thesamet
Copy link
Contributor

Bumping protobuf-java to 3.21.x would warrant a 0.12.x release for ScalaPB including a bunch of other libs, which I'd like to take more time for to consider additional changes/improvements in that release.

I believe the issue you are seeing is local to your project. ScalaPB doesn't use parseUknownField. Can you check if by any chance you are generating Java code using PB.protocVersion > 3.19.6? ScalaPB 0.11.x which is used by zio-grpc 0.5.x and 0.6.x relies on protobuf-java 3.19.6.

@regiskuckaertz
Copy link
Contributor Author

Oooh, ok I think I know what happened. Thanks for the pointers, I will have a look!

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.

2 participants