RFR: 8369950: TLS connection to IPv6 address fails with BCJSSE due to IllegalArgumentException [v2]
Jaikiran Pai
jpai at openjdk.org
Tue Dec 2 14:29:57 UTC 2025
On Tue, 2 Dec 2025 10:34:57 GMT, Sergey Chernyshev <schernyshev at openjdk.org> wrote:
>> Yes, doing this as a separate PR is fine with me.
>
> @jaikiran Let me clean up the comments that are not anymore accurate in this particular PR. I personally try to avoid this type of changes that only touch the whitespace or comments blocks, including the copyright header ones, because they increase the number of connections between unrelated changes, that makes the fix less portable.
@sercher, it's OK if you leave the test in the current form. Just fixing the `@summary` test on the test definition should be enough. I have spoken to others and the agreement is that since we already have similar tests in this area, it probably is a better thing to let this test stay in this manner instead of devicing some new way to test this. Sorry about the previous guidance to update these test comments.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28577#discussion_r2580623450
More information about the security-dev
mailing list