AxisHttpConnectionImpl.java throws an ioexception when the client closes the connection

[Nandika Jayawardana <ja...@gmail.com>]

Hi all,
In [1] when a client with keep-Alive setting is closing the connection, an
IO Exception is thrown, which is printed in the log as a warning message
with the exception. I checked the http core5 code as mentioned in the
method comment, and http core5 has decided to ignore this scenario and not
to throw an exception when the client closes the connection.

[1]
https://github.com/apache/axis-axis2-java-core/blob/master/modules/transport/http/src/main/java/org/apache/axis2/transport/http/server/AxisHttpConnectionImpl.java#L224
[2]
https://github.com/apache/httpcomponents-core/blob/5.3.x/httpcore5/src/main/java/org/apache/hc/core5/http/impl/io/DefaultBHttpServerConnection.java#L144
[3] https://issues.apache.org/jira/browse/HTTPCORE-677

Hence shall we change the code accordingly?

Regards
Nandika

Re: AxisHttpConnectionImpl.java throws an ioexception when the client closes the connection

[Nandika Jayawardana <ja...@gmail.com>]

Hi Robert,

Thank you for the response. I will try to send a PR.

Regards
Nandika

On Mon, 5 May 2025 at 19:40, robertlazarski <ro...@gmail.com>
wrote:

> Thanks for bringing this up. I didn't know about this.
>
> That axis2 class was the hardest part of the httpclient5 migration. It
> requires some time and frankly courage I don't have this week.
>
> Anyone on this list can submit a Github PR as this lib is what makes axis2
> possible more than just about anything else. It is used in very advanced
> ways not just in this class but in our testing framework also.
>
> On Wed, Apr 30, 2025 at 1:54 AM Nandika Jayawardana <ja...@gmail.com>
> wrote:
>
>> Hi all,
>> In [1] when a client with keep-Alive setting is closing the connection,
>> an IO Exception is thrown, which is printed in the log as a warning message
>> with the exception. I checked the http core5 code as mentioned in the
>> method comment, and http core5 has decided to ignore this scenario and not
>> to throw an exception when the client closes the connection.
>>
>> [1]
>> https://github.com/apache/axis-axis2-java-core/blob/master/modules/transport/http/src/main/java/org/apache/axis2/transport/http/server/AxisHttpConnectionImpl.java#L224
>> [2]
>> https://github.com/apache/httpcomponents-core/blob/5.3.x/httpcore5/src/main/java/org/apache/hc/core5/http/impl/io/DefaultBHttpServerConnection.java#L144
>> [3] https://issues.apache.org/jira/browse/HTTPCORE-677
>>
>> Hence shall we change the code accordingly?
>>
>> Regards
>> Nandika
>>
>

Re: AxisHttpConnectionImpl.java throws an ioexception when the client closes the connection

[robertlazarski <ro...@gmail.com>]

Thanks for bringing this up. I didn't know about this.

That axis2 class was the hardest part of the httpclient5 migration. It
requires some time and frankly courage I don't have this week.

Anyone on this list can submit a Github PR as this lib is what makes axis2
possible more than just about anything else. It is used in very advanced
ways not just in this class but in our testing framework also.

On Wed, Apr 30, 2025 at 1:54 AM Nandika Jayawardana <ja...@gmail.com>
wrote:

> Hi all,
> In [1] when a client with keep-Alive setting is closing the connection, an
> IO Exception is thrown, which is printed in the log as a warning message
> with the exception. I checked the http core5 code as mentioned in the
> method comment, and http core5 has decided to ignore this scenario and not
> to throw an exception when the client closes the connection.
>
> [1]
> https://github.com/apache/axis-axis2-java-core/blob/master/modules/transport/http/src/main/java/org/apache/axis2/transport/http/server/AxisHttpConnectionImpl.java#L224
> [2]
> https://github.com/apache/httpcomponents-core/blob/5.3.x/httpcore5/src/main/java/org/apache/hc/core5/http/impl/io/DefaultBHttpServerConnection.java#L144
> [3] https://issues.apache.org/jira/browse/HTTPCORE-677
>
> Hence shall we change the code accordingly?
>
> Regards
> Nandika
>