On Fri, 20 Nov 2020 20:12:47 GMT, Jamil Nimeh <jni...@openjdk.org> wrote:
>> src/java.base/share/classes/sun/security/ssl/SignatureScheme.java line 73: >> >>> 71: ED448 (0x0808, "ed448", "Ed448", >>> 72: "EdDSA", >>> 73: ProtocolVersion.PROTOCOLS_12_13), >> >> You may also want to check if EdDSA is available in the following block: >> - 282 if ("EC".equals(keyAlgorithm)) { >> + 282 if ("EC".equals(keyAlgorithm) || "EdDSA"... ) { >> 283 mediator = JsseJce.isEcAvailable(); >> 284 } > > JsseJce.isEcAvailable doesn't check for EdDSA availability so I'm not sure we > want that second clause. I don't think the EdDSA code is implemented in the > same module as the other EC code is so I don't know if we'd want to extend > isEcAvailable to include EdDSA. I'll need to go look to see where EdDSA is > located relative to the EC code. Hm, maybe a new isEdDsaAvailable(). ------------- PR: https://git.openjdk.java.net/jdk/pull/1197