<div dir="ltr"><div dir="ltr">On Tue, Jun 8, 2021 at 4:46 PM Michael Maier <<a href="mailto:m1278468@mailbox.org">m1278468@mailbox.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
On 08.06.21 at 10:48 Joshua C. Colp wrote:<br>
> On Mon, Jun 7, 2021 at 10:38 PM Michael Maier <<a href="mailto:m1278468@mailbox.org" target="_blank">m1278468@mailbox.org</a>> wrote:<br>
> <br>
>> On 06.06.21 at 22:19 Joshua C. Colp wrote:<br>
>>> On Sun, Jun 6, 2021 at 3:57 PM Michael Maier <<a href="mailto:m1278468@mailbox.org" target="_blank">m1278468@mailbox.org</a>><br>
>> wrote:<br>
>>><br>
>>>> Hello!<br>
>>>><br>
>>>> Using Asterisk 18.4 / pjisp, timers are advertised as supported though<br>
>>>> disabled in config with timers=no.<br>
>>>><br>
>>>> This does not happen initially (during the Invite sequence) but later on<br>
>>>> in 200 Ok as answer to a reInvite or as the answer to an Update methode.<br>
>>>><br>
>>>> Is there any reason why it's suddenly activated later on though it's<br>
>>>> deactivated? From my point of view, this smells like a bug.<br>
>>>><br>
>>><br>
>>> It'd be a bug in PJSIP itself, probably in the INVITE session[1] code>>> that is what responsible for this.<br>
>><br>
>> Thanks for your hint!<br>
>><br>
>> They are using a function to clean up the supported header<br>
>> (cleanup_allow_sup_hdr).<br>
>><br>
>> Let's take a look at the creation of the 200 Ok answer of the received<br>
>> update - I could find this path (there is no SDP in the received Update):<br>
>><br>
>> inv_respond_incoming_update<br>
>>          pjsip_dlg_create_response<br>
>>                  pjsip_endpt_create_response<br>
>>                          pjsip_msg_create<br>
>>                                  pj_list_init<br>
>><br>
>>          pjsip_timer_update_resp<br>
>>          pjsip_dlg_send_response<br>
>><br>
>> If I didn't oversee anything, I couldn't find the usage of<br>
>> cleanup_allow_sup_hdr - but I couldn't find either where the supported<br>
>> header should have been added. Do you have an idea?<br>
>><br>
> <br>
> It's added to the global Supported header, which is added elsewhere to<br>
> messages[1].<br>
<br>
dlg_beautify_response - I've seen it before but didn't take a closer look because of its name "beautify".<br>
<br>
My goal is now to add cleanup_allow_sup_hdr to pjsip_dlg_create_response after the response has been beautified ... .<br>
I figured out, that cleanup_allow_sup_hdr needs inv->options to know about the configuration of an endpoint. Unfortunately, this parameter isn't put to pjsip_dlg_create_response.<br>
Using the attached patch, I tried to do this, but I'm getting a compile error afterwards:<br>
<br>
make --quiet --no-print-directory -C pjproject all<br>
make --quiet --no-print-directory -C jansson all<br>
echo '[jansson] ' Building bundled jansson.<br>
(cd source; make >/dev/null 2>&1)<br>
(cd source; make install DESTDIR= >/dev/null 2>&1)<br>
echo '[pjproject] ' Compiling lib libpj-x86_64-unknown-linux-gnu.a<br>
make -C /home/test/rpmbuild-asterisk/BUILD/asterisk-18.4.0/third-party/pjproject/source/pjlib//build libpj-x86_64-unknown-linux-gnu.a >/dev/null 2>&1<br>
echo '[pjproject] ' Compiling lib libpjlib-util-x86_64-unknown-linux-gnu.a<br>
make -C /home/test/rpmbuild-asterisk/BUILD/asterisk-18.4.0/third-party/pjproject/source/pjlib-util//build libpjlib-util-x86_64-unknown-linux-gnu.a >/dev/null 2>&1<br>
make[2]: *** [/home/test/rpmbuild-asterisk/BUILD/asterisk-18.4.0/third-party/pjproject/source/pjlib-util/lib/libpjlib-util-x86_64-unknown-linux-gnu.a] Error 2<br>
make[1]: *** [pjproject] Error 2<br>
make: *** [third-party] Error 2<br>
<br>
Unfortunately, this isn't very verbose. Do you have any idea how to get it more verbose or maybe what should I probably additionally do to get it compiling? This is based on the<br>
bundled pjsip and the spec file of sangoma.<br></blockquote><div><br></div><div>I don't speak spec file but passing NOISY_BUILD=yes to make will generally output more information. </div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><font color="#073763">Joshua C. Colp</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Asterisk Technical Lead</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Sangoma Technologies</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Check us out at <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font><br></div></div></div></div></div></div></div></div></div></div></div>