Integrated: 8314978: Multiple server call from connection failing with expect100 in getOutputStream

[Vyom Tewari]

On Wed, 30 Aug 2023 09:23:34 GMT, Vyom Tewari <vtewari at openjdk.org> wrote:

> With the current implementation of HttpURLConnection  if server rejects the  “Expect 100-continue” then there will be ‘java.net.ProtocolException’ will be thrown from 'expect100Continue()' method. 
> 
> After the exception thrown, If we call any other method on the same instance (ex getHeaderField(), or getHeaderFields()). They will internally call getOuputStream() which invokes writeRequests(), which make the actual server call.
> 
> The code change will sets the existing variable ‘rememberedException’ when there is exception and getOutputStream0() will re-throw ‘rememberedException’  if the ‘rememberedException’ is not null. 
> 
> Note: getOutputStream0() also call’s  ‘expect100Continue()’  if ‘expectContinue’ is true.

This pull request has now been integrated.

Changeset: 460ebcd9
Author:    Vyom Tewari <vtewari at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/460ebcd9cb94867608e22e07092bd1cf33228700
Stats:     268 lines in 2 files changed: 268 ins; 0 del; 0 mod

8314978: Multiple server call from connection failing with expect100 in getOutputStream

Reviewed-by: dfuchs

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

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