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

Add support for ECDSA signatures using native OpenSSL library #54

Merged
merged 1 commit into from
May 2, 2024

Conversation

KostasTsiounis
Copy link
Contributor

A new class, namely NativeECDSASignature.java, is created and the corresponding options are added in other places to allow for the use of native OpenSSL library to create and verify ECDSA signatures.

Back-ported from: ibmruntimes/openj9-openjdk-jdk#783

Signed-off by: Kostas Tsiounis [email protected]

@KostasTsiounis
Copy link
Contributor Author

FYI @keithc-ca

@keithc-ca
Copy link
Member

Jenkins test sanity zlinux jdk22

@keithc-ca
Copy link
Member

All-too-common build error:

[2024-04-30T19:15:13.519Z] IOException caught during compilation: Resource deadlock avoided

@keithc-ca
Copy link
Member

Jenkins test sanity zlinux jdk22

@keithc-ca
Copy link
Member

Please squash in preparation for testing.

@KostasTsiounis
Copy link
Contributor Author

Commits squashed. Ready for testing.

@keithc-ca
Copy link
Member

Jenkins test sanity zlinux jdk22

@keithc-ca
Copy link
Member

keithc-ca commented May 2, 2024

@KostasTsiounis
Copy link
Contributor Author

Please address the type mentioned in https://github.com/ibmruntimes/openj9-openjdk-jdk/pull/783/files#r1588208020.

Fixed here too.

@keithc-ca
Copy link
Member

Jenkins test sanity zlinux jdk22

@keithc-ca
Copy link
Member

Grinder for jdk_security3 passed: https://openj9-jenkins.osuosl.org/job/Grinder/3563.

@keithc-ca keithc-ca merged commit 7b1ea2c into ibmruntimes:openj9 May 2, 2024
5 checks passed
@KostasTsiounis KostasTsiounis deleted the ecdsa_native branch May 16, 2024 18:25
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