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

jmigrate reports errors with org.conscrypt.Platform library #1171

Open
shenoyvvarun opened this issue Oct 2, 2023 · 1 comment
Open

jmigrate reports errors with org.conscrypt.Platform library #1171

shenoyvvarun opened this issue Oct 2, 2023 · 1 comment

Comments

@shenoyvvarun
Copy link

shenoyvvarun commented Oct 2, 2023

Summary

We (dependency from io.grpc) are currently using org.conscrypt:conscrypt-openjdk-uber:jar:2.5.2 and are facing errors with jdeps when running on JDK17.

/usr/bin/jdeps --ignore-missing-deps -jdkinternals

Class = org.conscrypt.Platform (Platform.java)
Location Line Number API Details Description
METHOD oidToAlgorithmName 547 CLASS sun.security.x509.AlgorithmId package sun.security.x509 is jdk internal after jdk 8. In jdk16, --illegal-access=deny was default. This option is ignored in jdk 17. module java.base does not export package sun.security.x509. usage context: sun.security.x509.AlgorithmId.get call (return type)
METHOD oidToAlgorithmName 547 CLASS sun.security.x509.AlgorithmId package sun.security.x509 is jdk internal after jdk 8. In jdk16, --illegal-access=deny was default. This option is ignored in jdk 17. module java.base does not export package sun.security.x509. usage context: sun.security.x509.AlgorithmId.get call
METHOD oidToAlgorithmName 547 CLASS sun.security.x509.AlgorithmId package sun.security.x509 is jdk internal after jdk 8. In jdk16, --illegal-access=deny was default. This option is ignored in jdk 17. module java.base does not export package sun.security.x509. usage context: sun.security.x509.AlgorithmId.getName call
@prbprbprb
Copy link
Collaborator

Yeah, we'll have to get rid of that dependency but it would be a breaking change.

I guess the solution is to expand our own OidData class enough that the Sun class is no longer needed. However our class is Apache licensed and the Sun one is GPL, so we'd have to do some kind of cleanroom development to achieve that in a legal way.

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

No branches or pull requests

2 participants