camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thomas Papke (JIRA)" <>
Subject [jira] [Updated] (CAMEL-9739) Mina2Consumer exception handler do close session also for IOException
Date Tue, 22 Mar 2016 12:03:25 GMT


Thomas Papke updated CAMEL-9739:
    Attachment: IoException.patch

> Mina2Consumer exception handler do close session also for IOException
> ---------------------------------------------------------------------
>                 Key: CAMEL-9739
>                 URL:
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-mina2
>    Affects Versions: 2.16.2
>            Reporter: Thomas Papke
>         Attachments: IoException.patch
> The camel-mina2 consumer Exception handler ({{org.apache.camel.component.mina2.Mina2Consumer.ReceiveHandler.exceptionCaught(IoSession,
> Is an implementation of the Mina2 interface {{org.apache.mina.core.service.IoHandler}}.
The Javadoc Mina2 documentation of exceptionCaught explicitly mention, that Mina2 will close
the connection for all IOExceptions automatically. But camel-mina2 seems to close the connection
for any kind of exception.
> *Current behavior:* If an error occurs on the transport layer (e.g. secure TCP connection
where the certificate was not trusted and a SSLException occure), I saw the following warning:
> {code}
> org.apache.camel.component.mina2.Mina2Consumer#exceptionCaught(376) - Closing session
as an exception was thrown from MINA
> {code}
> And also an error from mina2:
> {code}
> org.apache.mina.filter.ssl.SslHandler#destroy(210) - Unexpected exception from SSLEngine.closeInbound().
> " Inbound closed before receiving peer's close_notify: possible
truncation attack?
> {code}
> From a functional perspective, beside the annoying log entries, it looks like the SSL
error details are sometime not send properly back to the client.
> *Current workaround:* I don’t have one.
> *Suggested bugfix:* Implement the Mina2Consumer exception handling similar to {{,
Throwable)}} and do not handle IoException and simply rethrow them. E.g.
> {code}
> if (cause instanceof IOException) {
>"IOException will be closed by Mina itself")
> 	return;
> }
> {code}

This message was sent by Atlassian JIRA

View raw message