<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6807">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I do not need this and I do not think this is good behavior of an UAS. For me, keep-alive kills TCP/TLS connections. A default should not crash a remote phone. Some platforms like Apple iOS might even close such chatty connections. I would have to double-check that. This is not something an UAS should do because the UAS is behind a NAT normally. Finally, chan_sip did not do that either on default.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This might be interesting for Asterisk as UAC. However, if I am not mistaken even that runtime-setting introduced with Asterisk 13.2.0 is not that fine grained – it is for all connections, isn’t it? There is no way to limit this to IPv4/UAS in general or a specific endpoint, or? The original commit message of that change in Asterisk 13.2.0 mentioned something like 33 (!) seconds for UACs. I was not able to find something like that in the source code of the PJProject, yet. Somebody else should investigate that, if you really want it to match the previous chan_pjsip behavior. Again, chan_sip does not do this on default either.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I do not use CRLN based keep-alives for any TCP/TLS connections. If you want it, please, hijack this change, go for DEFAULT_KEEPALIVE_INTERVAL = 90, and change sip_to_pjsip.py. I went for matching the default behavior of chan_sip.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6807">change 6807</a>. To unsubscribe, 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/6807"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I27c7a87c882b02f56ff3b739630d9b9e5912cc68 </div>
<div style="display:none"> Gerrit-Change-Number: 6807 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 16 Oct 2017 11:37:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>