<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This change seems correct to me so I'll provide a +1 for code review. I will defer to the community two questions though:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Firstly, is there a correct way to specify the contact header as a library user of sofia-sip? I am not sure if having the library infer the contact header is the ideal behavior, since this functionality was removed in freeswitch sofia-sip years ago and has not been raised there as an issue. If the contact header is being included in the BYE message against the RFC, is that an additional bug which should be filed, or did I not specify the contact header correctly? As stated in my original patch I am not a sip expert, so I will defer to the collective experience of the community here.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Secondly, I am not sure when changes will be made to freeswitch's library. Reverting this patch now will make osmo-sip-connector and freeswitch's libsofia-sip incompatible again. I can raise this issue in the freeswitch slack and on github again, but I am not sure when it will be addressed. It may be worth considering holding on merging this revert until there is a clear path forward on that end. I can patch my local build, but anyone else using freeswitch with osmo-sip-connector will run into the same problem.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It seems like I currently do not have permissions to edit the code review label yet, but have reviewed the change and commit message.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/19992">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/19992">change 19992</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/c/osmo-sip-connector/+/19992"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sip-connector </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I712f17fecbc372d1e486e80673a548e281b37800 </div>
<div style="display:none"> Gerrit-Change-Number: 19992 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: matt9j <matt9j@cs.washington.edu> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 05 Sep 2020 23:20:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>