> Per [RFC 6066 "3. Server Name Indication"], disallow IP literals in 
> `SNIHostName::new`.
> 
> While the following two call-sites could be simplified by removing IP literal 
> checks, I've refrained from doing so because delegating some of the checks to 
> an exception catching mechanism would impact the performance:
> 
>     sun.security.ssl.Utilities::rawToSNIHostName
>     sun.net.www.protocol.https.HttpsClient::afterConnect
> 
> [RFC 6066 "3. Server Name Indication"]: 
> https://www.rfc-editor.org/rfc/rfc6066.html#page-6
> 
> ---------
> - [X] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Volkan Yazici has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains 19 additional commits 
since the last revision:

 - Merge remote-tracking branch 'upstream/master' into sni
 - Ensure `encoded` is copied
 - Add tests and docs for UTF-8 vs. ASCII discrepancy
 - Remove `{@return` tags
 - Add reference to RFC 5280
 - Simplify "ASCII or UTF-8" with just "UTF-8"
 - Revert javadoc changes from deprecated methods
 - Do the forgotten update
 - Improve wording and HTML rendering
 - Merge remote-tracking branch 'upstream/master' into sni
 - ... and 9 more: https://git.openjdk.org/jdk/compare/5baa1c14...0e562368

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/30747/files
  - new: https://git.openjdk.org/jdk/pull/30747/files/abbb3a3e..0e562368

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=30747&range=11
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30747&range=10-11

  Stats: 17419 lines in 330 files changed: 12660 ins; 2385 del; 2374 mod
  Patch: https://git.openjdk.org/jdk/pull/30747.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/30747/head:pull/30747

PR: https://git.openjdk.org/jdk/pull/30747

Reply via email to