-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
Exclusing DisabledCurve on z/OS #4457
Conversation
@llxia @Mesbah-Alam Could you please review this PR. |
Verification job excluded the test |
@@ -291,7 +291,8 @@ sun/security/provider/SecureRandom/StrongSecureRandom.java https://github.ibm.co | |||
sun/security/rsa/PrivateKeyEqualityTest.java https://github.ibm.com/runtimes/backlog/issues/795 macosx-all | |||
sun/security/rsa/pss/SignatureTest2.java https://github.ibm.com/runtimes/backlog/issues/795 macosx-all | |||
sun/security/rsa/pss/SignatureTestPSS.java https://github.ibm.com/runtimes/backlog/issues/795 macosx-all | |||
sun/security/ssl/CipherSuite/DisabledCurve.java https://github.ibm.com/runtimes/backlog/issues/795 aix-all,windows-all | |||
#sun/security/ssl/CipherSuite/DisabledCurve.java is excluded as Invalid scenario after removing unsupported binary curves from SSLSocketTemplate.java |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we add a reference to
Permanent excludes issue #1297 and add comments in the issue as well? Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added this test in Permanent excludes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @psoujany
This PR is to exclude sun/security/ssl/CipherSuite/DisabledCurve.java on z/OS