You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Robbie Gemmell (Jira)" <ji...@apache.org> on 2019/11/28 09:59:00 UTC

[jira] [Resolved] (PROTON-2142) transports closed due to decode error may emit a close frame without error condition

     [ https://issues.apache.org/jira/browse/PROTON-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robbie Gemmell resolved PROTON-2142.
------------------------------------
    Resolution: Fixed

> transports closed due to decode error may emit a close frame without error condition
> ------------------------------------------------------------------------------------
>
>                 Key: PROTON-2142
>                 URL: https://issues.apache.org/jira/browse/PROTON-2142
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: proton-j
>    Affects Versions: proton-j-0.33.2
>            Reporter: Robbie Gemmell
>            Assignee: Robbie Gemmell
>            Priority: Major
>             Fix For: proton-j-0.33.3
>
>
> When a decode error occurs while processing new input, the related transport is closed with the exception, which sets an error condition on the transport. Upon further processing of output, the transport emits a close frame.
> If the connection endpoint is bound to the transport at this point, the error condition object from the connection endpoint is used when generating the close frame, even if it contains no error detail (which it doesn't, unless explicitly set by the using code). The close frame emitted which is then emitted has no error condition, giving no indication to the peer (or those debugging the protocol interactions) of there being a problem or what cause it. In this case, sending detail from the transport error condition would be far more useful.
> Since the transport and connection form an overall unit in use, the handling should be updated to use the connection error condition only if its still bound *and populated*, but to otherwise use the transport error condition if set.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org