You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2021/05/06 19:13:00 UTC

[jira] [Commented] (PROTON-2380) frame tracing issues/improvements

    [ https://issues.apache.org/jira/browse/PROTON-2380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17340378#comment-17340378 ] 

ASF subversion and git services commented on PROTON-2380:
---------------------------------------------------------

Commit bb2d2de973c6aa268f367deac2bdee48a5710130 in qpid-protonj2's branch refs/heads/main from Timothy Bish
[ https://gitbox.apache.org/repos/asf?p=qpid-protonj2.git;h=bb2d2de ]

PROTON-2380 Show difference of SASL vs AMQP default header in logs

> frame tracing issues/improvements
> ---------------------------------
>
>                 Key: PROTON-2380
>                 URL: https://issues.apache.org/jira/browse/PROTON-2380
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: protonj2
>    Affects Versions: protonj2-1.0.0-M1
>            Reporter: Robbie Gemmell
>            Assignee: Timothy A. Bish
>            Priority: Major
>
> Running the HelloWorld example against Broker-J 8.0.4 with PN_TRACE_FRM=true style frame trace enabled shows some areas for improvement.
> * Nothing distinguishes the connection it applies to (e.g in other cases we log include the memory address), limiting value in multi-connection use cases. 
> * The run ultimately resulted in exposing a broker bug (QPID-8523), causing connection closure with explanation, but the attach frame trace itself didnt show the issue at all, instead implying a value was sent (printing initialDeliveryCount=0). Some way of distinguishing actual vs default (in code terms) values received, and possibly how many fields were populated, would be useful. 
> * Some binary bits will just print out the buffer class/sizes etc.
> * The initial SASL header is prefixed 'AMQP' which is debatable (both ways, since its only a header). May be simpler to say SASL to show transition. Or drop the prefix.



--
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