<div dir="ltr">Thanks, regardless I am glad somebody took care of refactoring that old code, but please be more careful with merging it down into stable branches. :) I've just made a change to request a daily build of the voiptests, so consider keeping an eye on it after making changes in master branch.<div><br></div><div>It is just happens you caught me in the process of merging python 3.x support in the sippy so it's a bit of an unexpected obstacle for us here.<br><div><br></div><div>-Max</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 5, 2018 at 12:24 PM, Ovidiu Sas <span dir="ltr"><<a href="mailto:osas@voipembedded.com" target="_blank">osas@voipembedded.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Maxim,<br>
<br>
I think I know what the issue is. There was a confusion between<br>
variable naming: oldpf and newpf refers to protocol format and not the<br>
actual IP.<br>
I will revert the changes!<br>
<br>
-ovidiu<br>
<div class="HOEnZb"><div class="h5"><br>
On Mon, Feb 5, 2018 at 3:06 PM, Maxim Sobolev <<a href="mailto:sobomax@sippysoft.com">sobomax@sippysoft.com</a>> wrote:<br>
> That merge broke all of our OpenSIPS test scenarous on voiptests. We are<br>
> investigating about the possible cause but I think this change may need to<br>
> be reverted from the stable branches until it's clear what's going on.<br>
><br>
> -Maxim<br>
><br>
> On Mon, Feb 5, 2018 at 10:04 AM, Ovidiu Sas <<a href="mailto:osas@voipembedded.com">osas@voipembedded.com</a>> wrote:<br>
>><br>
>>   Branch: refs/heads/2.3<br>
>>   Home:   <a href="https://github.com/OpenSIPS/opensips" rel="noreferrer" target="_blank">https://github.com/OpenSIPS/<wbr>opensips</a><br>
>>   Commit: a45d4dcaf046bb273cfe5905ac0358<wbr>45a6867945<br>
>><br>
>> <a href="https://github.com/OpenSIPS/opensips/commit/a45d4dcaf046bb273cfe5905ac035845a6867945" rel="noreferrer" target="_blank">https://github.com/OpenSIPS/<wbr>opensips/commit/<wbr>a45d4dcaf046bb273cfe5905ac0358<wbr>45a6867945</a><br>
>>   Author: Ovidiu Sas <<a href="mailto:osas@voipembedded.com">osas@voipembedded.com</a>><br>
>>   Date:   2018-02-05 (Mon, 05 Feb 2018)<br>
>><br>
>>   Changed paths:<br>
>>     M modules/nathelper/nathelper.c<br>
>><br>
>>   Log Message:<br>
>>   -----------<br>
>>   nathelper: manual backport for 3c39167e333a8801772f49f8561a01<wbr>5bfa1836f1<br>
>>  - fix test condition for updateing IP for fix_nated_sdp()<br>
>><br>
>><br>
>><br>
>> ______________________________<wbr>_________________<br>
>> Devel mailing list<br>
>> <a href="mailto:Devel@lists.opensips.org">Devel@lists.opensips.org</a><br>
>> <a href="http://lists.opensips.org/cgi-bin/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.opensips.org/cgi-<wbr>bin/mailman/listinfo/devel</a><br>
>><br>
><br>
><br>
><br>
> --<br>
> Maksym Sobolyev<br>
> Sippy Software, Inc.<br>
> Internet Telephony (VoIP) Experts<br>
> Tel (Canada): <a href="tel:%2B1-778-783-0474" value="+17787830474">+1-778-783-0474</a><br>
> Tel (Toll-Free): <a href="tel:%2B1-855-747-7779" value="+18557477779">+1-855-747-7779</a><br>
> Fax: <a href="tel:%2B1-866-857-6942" value="+18668576942">+1-866-857-6942</a><br>
> Web: <a href="http://www.sippysoft.com" rel="noreferrer" target="_blank">http://www.sippysoft.com</a><br>
> MSN: <a href="mailto:sales@sippysoft.com">sales@sippysoft.com</a><br>
> Skype: SippySoft<br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
VoIP Embedded, Inc.<br>
<a href="http://www.voipembedded.com" rel="noreferrer" target="_blank">http://www.voipembedded.com</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Maksym Sobolyev<br>Sippy Software, Inc.<br>Internet Telephony (VoIP) Experts<br>Tel (Canada): +1-778-783-0474<br>Tel (Toll-Free): +1-855-747-7779<br>Fax: +1-866-857-6942<br>Web: <a href="http://www.sippysoft.com" target="_blank">http://www.sippysoft.com</a><br>MSN: <a href="mailto:sales@sippysoft.com" target="_blank">sales@sippysoft.com</a><br>Skype: SippySoft<br></div></div>
</div>