<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13726">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13726/3/main/stasis_channels.c">File main/stasis_channels.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/13726/3/main/stasis_channels.c@1361">Patch Set #3, Line 1361:</a> <code style="font-family:monospace,monospace">   res |= ast_json_object_set(to_json, "asterisk_id", ast_json_string_create(</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure if this is the correct place for this? It seems like somewhere further down the return path might better. That way it would capture all messages (as you state some seemingly don't contain the asterisk_id).</p><p style="white-space: pre-wrap; word-wrap: break-word;">That said I do see in res/stasis/app.c - app_send(...) where the asterisk_id is being set. Which seems like a good place to do it as it adds it to all messages. I wonder why it is not being added for messages like this one? It seems like it should be. Are you seeing a warning of any kind: "Failed to append EID ..."?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/13726">change 13726</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/13726"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I85fd92c21e7c3465059acda34adf6cfe1777147d </div>
<div style="display:none"> Gerrit-Change-Number: 13726 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Sylvain Afchain <safchain@wazo.io> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 05 Feb 2020 23:50:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>