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

Java 11 warning for protobuf illegal reflective access #62

Closed
antoniy opened this issue Nov 2, 2018 · 4 comments · Fixed by #152
Closed

Java 11 warning for protobuf illegal reflective access #62

antoniy opened this issue Nov 2, 2018 · 4 comments · Fixed by #152

Comments

@antoniy
Copy link

antoniy commented Nov 2, 2018

We've doing a migration of our application to Java 11 and Spring Boot 2.1.0 and we get these warning messages in our logs:

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.github.os72.protobuf351.UnsafeUtil (file:/Users/user/.m2/repository/com/signalfx/public/signalfx-java/0.0.48/signalfx-java-0.0.48.jar) to field java.nio.Buffer.address
WARNING: Please consider reporting this to the maintainers of com.github.os72.protobuf351.UnsafeUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Not sure if this repository is the right one to reise the issue but I considered that we are using signalfx library and not protobuf so here should be the best place to report the issue.

@mpetazzoni
Copy link
Contributor

I think that would be something to take up with the os72 shaded protobuf libraries that we use (https://github.com/os72) ?

We haven't looked at Java 11 support yet for signalfx-java.

@finci-square
Copy link

+1 I'm using this in java 11

@MatthewRyanRead
Copy link

MatthewRyanRead commented May 17, 2021

@mpetazzoni Protobuf fixed this in 3.7.0 according to protocolbuffers/protobuf#5549, and there is a shaded version of 3.11.1 at https://github.com/os72/protobuf-java-shaded-3-11-1. So I think signalfx-java can just bump these versions now (although there might be issues to resolve).

@jessicadeery
Copy link

+1 for this.

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 a pull request may close this issue.

5 participants