[jdk17u-dev] RFR: 8272702: Resolving URI relative path with no / may lead to incorrect toString

[Radim Vansa]

On Tue, 21 Feb 2023 14:17:39 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this as it affects to JDK 17.
>> Does not apply clean due to a conflict with https://bugs.openjdk.org/browse/JDK-8285521 but it's the same logic.
>> 
>> ---------
>> ### Progress
>> - [ ] Change must be properly reviewed (1 review required, with at least 1 [Reviewer](https://openjdk.org/bylaws#reviewer))
>> - [x] Change must not contain extraneous whitespace
>> - [x] Commit message must refer to an issue
>> 
>> ### Error
>>  ⚠️ The pull request body must not be empty.
>> 
>> 
>> 
>> ### Reviewing
>> <details><summary>Using <code>git</code></summary>
>> 
>> Checkout this PR locally: \
>> `$ git fetch https://git.openjdk.org/jdk17u-dev pull/1176/head:pull/1176` \
>> `$ git checkout pull/1176`
>> 
>> Update a local copy of the PR: \
>> `$ git checkout pull/1176` \
>> `$ git pull https://git.openjdk.org/jdk17u-dev pull/1176/head`
>> 
>> </details>
>> <details><summary>Using Skara CLI tools</summary>
>> 
>> Checkout this PR locally: \
>> `$ git pr checkout 1176`
>> 
>> View PR using the GUI difftool: \
>> `$ git pr show -t 1176`
>> 
>> </details>
>> <details><summary>Using diff file</summary>
>> 
>> Download this PR as a diff file: \
>> <a href="https://git.openjdk.org/jdk17u-dev/pull/1176.diff">https://git.openjdk.org/jdk17u-dev/pull/1176.diff</a>
>> 
>> </details>
>
> How did you test this? Does the test pass with the fix?

@GoeLin Yes, I tested this through `make test TEST=test/jdk/java/net/`:


==============================
Test summary
==============================
   TEST                                              TOTAL  PASS  FAIL ERROR   
   jtreg:test/jdk/java/net                             595   595     0     0   
==============================
TEST SUCCESS

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

PR: https://git.openjdk.org/jdk17u-dev/pull/1176