RFR: 8276559: (httpclient) Consider adding an HttpRequest.Builder.HEAD method to build a HEAD request. [v2]

Jaikiran Pai jpai at openjdk.java.net
Wed Nov 17 03:25:38 UTC 2021


On Fri, 12 Nov 2021 08:01:50 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with four additional commits since the last revision:
>> 
>>  - Implement Daniel's review suggestion - use a different URI for HEAD() request in HttpRequestNewBuilderTest
>>  - Remove unused imports and fields from HeadTest
>>  - - Update the HeadTest to use the convenience request builder methods GET() and HEAD()
>>    - Also add @bug tag to updated tests
>>  - fix comment about special casing certain HTTP methods
>
> LGTM

Thank you Daniel and @sormuras for the reviews.

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

PR: https://git.openjdk.java.net/jdk/pull/6348


More information about the net-dev mailing list