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

ASF GitHub Bot commented on PROTON-1998:
----------------------------------------

tabish121 commented on issue #30: PROTON-1998: Add SASL protocol trace
URL: https://github.com/apache/qpid-proton-j/pull/30#issuecomment-459768206
 
 
   I'd tend to agree with @gemmellr that the change should be more targeted and 
not alter the TransportFrame at this point.  Also I'm wondering is we want to 
default the not logging SASL frames unless configured to as it does expose that 
data where it was not doing so before which could be surprising to some.  It 
probably isn't a huge deal as the bytes can be seen anyway and possibly logged 
by other frameworks...
   
   I guess proton-c just logs all of it now so being consistent isn't a 
terrible thing.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> [Proton-J] Add SASL protocol trace
> ----------------------------------
>
>                 Key: PROTON-1998
>                 URL: https://issues.apache.org/jira/browse/PROTON-1998
>             Project: Qpid Proton
>          Issue Type: Improvement
>          Components: proton-j
>            Reporter: Keith Wall
>            Priority: Minor
>
> Unlike Proton, Proton-J does not provide SASL frame trace if environment 
> variable PN_TRACE_FRM is set.  It would be useful if Proton-J had this 
> ability too to help diagnose SASL negotiation problem.
> Proton's SASL frame trace looks like this:
> {code:java}
> [0x7fc112c03a00]: -> SASL
> [0x7fc112c03a00]: <- SASL
> [0x7fc112c03a00]:0 <- @sasl-mechanisms(64) 
> [sasl-server-mechanisms=@PN_SYMBOL[:ANONYMOUS]]
> [0x7fc112c03a00]:0 -> @sasl-init(65) [mechanism=:ANONYMOUS, 
> initial-response=b"[email protected]"]
> [0x7fc112c03a00]:0 <- @sasl-outcome(68) [code=0]
> [0x7fc112c03a00]: -> AMQP
> [0x7fc112c03a00]:0 -> @open(16) 
> [container-id="be777c26-af6e-4935-a6be-316cc8bbdb35", hostname="127.0.0.1", 
> channel-max=32767]{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to