Code review request, JDK-8017157, catch more exception in test RejectClientRenego (original) (raw)
Vincent Ryan vincent.x.ryan at oracle.com
Thu Jun 20 14:17:38 UTC 2013
- Previous message (by thread): Code review request, JDK-8017157, catch more exception in test RejectClientRenego
- Next message (by thread): quick question - bug_id 8014618
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Your change looks fine to me.
On 20 Jun 2013, at 10:42, Xuelei Fan wrote:
Hi,
Please review this test only update: http://cr.openjdk.java.net/~xuelei/8017157/webrev.00/ In test RejectClientRenego.java, we tried to catch SSLHandshakeException in both client and server. However, the client thread may throw SocketException when it receives server handshake failure messages. This update loosens the exception check by ignoring IOException rather than SSLHandshakeException: - } catch (SSLHandshakeException she) { + } catch (IOException ioe) { Thanks, Xuelei
- Previous message (by thread): Code review request, JDK-8017157, catch more exception in test RejectClientRenego
- Next message (by thread): quick question - bug_id 8014618
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]