-
Notifications
You must be signed in to change notification settings - Fork 235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Logging applied to more areas #27
Conversation
Logging applied to more areas
Logging applied to more areas
Hi @azmigproject, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
Please add more logging in -
|
Updated logs per feedback
Indentation corrected
@@ -354,15 +354,17 @@ public Integer sendMessage(Message message) | |||
byte[] tag = String.valueOf(this. nextTag++).getBytes(); | |||
Delivery dlv = sender.delivery(tag); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add logging on line 345 just before breaking that encoding is complete ? I suspect the issue is somewhere in this block.
Also, can you please sign the CLA for this request to get in. |
Please sign CLA before merging.
logging added to message encoding area
Hi, I am closing and re-opening this PR to bump the CLA bot. Sorry for the inconvenience! |
Hi @azmigproject, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
No description provided.