RFR: 8257725: No throws of SSLHandshakeException
Xue-Lei Andrew Fan
xuelei at openjdk.java.net
Thu Dec 3 21:03:08 UTC 2020
In the StatusResponseManager.get() method spec, the SSLHandshakeException is declared as throws exception. However, no such checked-exception would be thrown in the method implementation.
/**
...
* @throws SSLHandshakeException if an unsupported
* {@code CertStatusRequest} is provided.
*/
Map<X509Certificate, byte[]> get(CertStatusRequestType type,
CertStatusRequest request, X509Certificate[] chain, long delay,
TimeUnit unit) {
As the exception is a checked-exception, and is not declared in the method, it is safe to remove the throws spec.
Cleanup only, no new regression test.
Bug: https://bugs.openjdk.java.net/browse/JDK-8257725
-------------
Commit messages:
- 8257725: No throws of SSLHandshakeException
- 8257724: Incorrect package of the linked class in BaseSSLSocketImpl
Changes: https://git.openjdk.java.net/jdk/pull/1607/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1607&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8257725
Stats: 7 lines in 2 files changed: 1 ins; 3 del; 3 mod
Patch: https://git.openjdk.java.net/jdk/pull/1607.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1607/head:pull/1607
PR: https://git.openjdk.java.net/jdk/pull/1607
More information about the security-dev
mailing list