Nice work Ivan.
I see you've some clean up done on exception handling also. I might have
a concern on this change in SSLLogger. You're catching IOException now
instead of Exception. Given that it's a logger and the intent seems to
be to ignore any type of exception - should we leave what's there ?
Everything else looked fine to me.
Regards,
Sean.
On 10/11/19 09:47, Ivan Gerasimov wrote:
Hello!
There are many places in the security libraries where string are
converted from/to byte arrays using standard charsets.
It would be beneficial, if those are not looked up by their name every
time, and constants from java.nio.charset.StandardCharsets.* are used
instead.
Would you please help review this relatively lengthy (in terms of the
affected files), though mostly straight-forward cleanup?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8233884
WEBREV: http://cr.openjdk.java.net/~igerasim/8233884/00/webrev/
Thanks in advance!