On Thu, 12 May 2022 12:23:34 GMT, Jaikiran Pai wrote:
>> Will await further review for a short while and then integrate if approved.
>
>> Will await further review for a short while and then integrate if approved.
>
> Hello Conor, looking at the latest state of the PR, I think you might have
>
On Thu, 12 May 2022 12:23:34 GMT, Jaikiran Pai wrote:
>> Will await further review for a short while and then integrate if approved.
>
>> Will await further review for a short while and then integrate if approved.
>
> Hello Conor, looking at the latest state of the PR, I think you might have
>
On Fri, 6 May 2022 13:46:41 GMT, Conor Cleary wrote:
>> **Issue**
>> When using the `HttpClient.send()` to send a GET request created using the
>> `HttpRequest.newBuilder()`, a `Content-length: 0` header is set. This
>> behaviour causes issues with many services as a body related header is
>>
On Thu, 12 May 2022 12:01:23 GMT, Conor Cleary wrote:
> Will await further review for a short while and then integrate if approved.
Hello Conor, looking at the latest state of the PR, I think you might have
missed Daniel's review comment
https://github.com/openjdk/jdk/pull/8017#discussion_r866
On Fri, 6 May 2022 13:46:41 GMT, Conor Cleary wrote:
>> **Issue**
>> When using the `HttpClient.send()` to send a GET request created using the
>> `HttpRequest.newBuilder()`, a `Content-length: 0` header is set. This
>> behaviour causes issues with many services as a body related header is
>>
> **Issue**
> When using the `HttpClient.send()` to send a GET request created using the
> `HttpRequest.newBuilder()`, a `Content-length: 0` header is set. This
> behaviour causes issues with many services as a body related header is
> usually not expected to be included with a GET request.
>
