Integrated: 8257725: No throws of SSLHandshakeException
Xue-Lei Andrew Fan
xuelei at openjdk.java.net
Fri Dec 4 18:33:27 UTC 2020
On Thu, 3 Dec 2020 20:58:55 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:
> 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
This pull request has now been integrated.
Changeset: d76039d3
Author: Xue-Lei Andrew Fan <xuelei at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/d76039d3
Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
8257725: No throws of SSLHandshakeException
Reviewed-by: jnimeh
-------------
PR: https://git.openjdk.java.net/jdk/pull/1607
More information about the security-dev
mailing list