You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by GitBox <gi...@apache.org> on 2021/12/16 03:40:38 UTC
[GitHub] [qpid-proton] astitcher opened a new pull request #347: Extend frame logging to optionally decode transfer payloads as AMQP
astitcher opened a new pull request #347:
URL: https://github.com/apache/qpid-proton/pull/347
You can turn on this extra frame logging by setting the 'frame+' log level. This can usually be done by setting the PN_LOG environment variable to 'frame+'
This PR also contains changes to the recent AMQP value dump code so that it uses a fixed size string output buffer instead of using the pn_string_t type which will grow indefinitely.
I'd be very interested in people using this extra frame trace to see whether they think it's useful.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-proton] astitcher commented on pull request #347: Extend frame logging to optionally decode transfer payloads as AMQP
Posted by GitBox <gi...@apache.org>.
astitcher commented on pull request #347:
URL: https://github.com/apache/qpid-proton/pull/347#issuecomment-1000641429
In response to some feedback I've made the full frame logging the default (and only) behaviour when frame tracing is turned on (PN_LOG=frame). The extra control wasn't very useful, at least for this proof of concept.
Now that null values are not printed in frame traces the traces become significantly less cluttered and more useful than before.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-proton] astitcher commented on pull request #347: Extend frame logging to optionally decode transfer payloads as AMQP
Posted by GitBox <gi...@apache.org>.
astitcher commented on pull request #347:
URL: https://github.com/apache/qpid-proton/pull/347#issuecomment-1000641981
My intention is to split out the fixed string work from this branch as it isn't really the focus of this WIP, but an independently useful change which gos to bounding the size of the logging buffers that be received by a logger sink.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org