[Kamailio-Users] SDP ending with CRLF and a=nortpproxy:yes
Daniel-Constantin Mierla
miconda at gmail.com
Mon Sep 29 12:12:29 CEST 2008
On 09/28/08 12:44, Schober Walter wrote:
> I see: * [r4968] modules/nathelper/nathelper.c:
> - fix the calculation of CRLF length when adding norttptoxy string
> - patch by Alex Hermann (closes #2117700)
>
> And yes, it's that issue and therefore fixed in 1.4.1.
>
Indeed this fix is included in 1.4.1, but the original question to add
or not the line after m= line or at global level exists.
Cheers,
Daniel
>
>> -----Ursprüngliche Nachricht-----
>> Von: Schober Walter
>> Gesendet: Samstag, 27. September 2008 14:13
>> An: 'miconda at gmail.com'; Jerome Martin
>> Cc: users at lists.kamailio.org
>> Betreff: AW: [Kamailio-Users] SDP ending with CRLF and
>> a=nortpproxy:yes
>>
>> Hi!
>>
>> Same issue with 1.4.0. I'm migrating from 1.1.1 to 1.4.0, but
>> force_rtp_proxy() now adds a sdp line with a beginning 0x00,
>> so line looks like
>> 0000 00 61 3d 6e 6f 72 74 70 70 72 6f 78 79 3a 79 65
>> .a=nortpproxy:ye
>> 0010 73 s
>> But no \r\n\r\n in front anymore in 1.4.0.
>>
>> Someone to direct me to the propper source file,pls?
>>
>> Walter
>>
>>
>>> -----Ursprüngliche Nachricht-----
>>> Von: users-bounces at lists.kamailio.org
>>> [mailto:users-bounces at lists.kamailio.org] Im Auftrag von
>>> Daniel-Constantin Mierla
>>> Gesendet: Montag, 15. September 2008 14:23
>>> An: Jerome Martin
>>> Cc: users at lists.kamailio.org
>>> Betreff: Re: [Kamailio-Users] SDP ending with CRLF and
>>> a=nortpproxy:yes
>>>
>>> Hi Jerome,
>>>
>>> On 09/15/08 13:43, Jerome Martin wrote:
>>>
>>>> Hi All,
>>>>
>>>> Just FYI, I had to disable the nortpproxy test here because
>>>>
>>> it breaks
>>>
>>>> thomson ST2030 endpoints : with that line, they would just plain
>>>> ignore the messages containing the SDP ...
>>>>
>>> was i with malformed SDP (empty lines inside SDP before the
>>>
>> a= line),
>>
>>> or just the presence of the a=nortpproxy:yes line made the
>>>
>> ST2030 go
>>
>>> mad?
>>>
>>> Cheers,
>>> Daniel
>>>
>>>
>>>> Cheers,
>>>>
>>>> On Fri, 2008-09-12 at 18:12 +0300, Daniel-Constantin Mierla wrote:
>>>>
>>>>> On 09/11/08 16:10, Klaus Darilion wrote:
>>>>>
>>>>>> A similar bug was also described in opensips. They tried
>>>>>>
>>> to fix it
>>>
>>>>>> but the bug is still present. I wonder if this is
>>>>>>
>>> something new or
>>>
>>>>>> if the bug is thre since the beginning of nathelper
>>>>>>
>> but did not
>>
>>>>>> occured until now
>>>>>>
>>>>>>
>>>>> maybe this can be fixed by using the sdp parser available
>>>>>
>>> now in core.
>>>
>>>>> nathelper has quite limited sdp parsing capabilities now.
>>>>>
>>> the issue
>>>
>>>>> is to find the proper hook to add the lump with
>>>>>
>>>>> a=nortpproxy:yes\r\n"
>>>>>
>>>>> Cheers,
>>>>> Daniel
>>>>>
>>>>>
>>>>>
>>>>>> klaus
>>>>>>
>>>>>> Aymeric Moizard schrieb:
>>>>>>
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> I just happen to find an issue with a software trying to call
>>>>>>> through my openser-1.3.1
>>>>>>>
>>>>>>> After inserting "a=nortpproxy:yes\r\n", the message
>>>>>>>
>> coming out
>>
>>>>>>> openser is:
>>>>>>>
>>>>>>> [normal sdp packet]
>>>>>>> a=fmtp:101 0-15\r\n
>>>>>>> \r\n
>>>>>>> \r\n
>>>>>>> "a=nortpproxy:yes\r\n"
>>>>>>>
>>>>>>> No matter who is responsible in that specific case (the SDP
>>>>>>> received by openser with several \r\n looks not
>>>>>>>
>>> compliant to me).
>>>
>>>>>>> I think openser should add "a=nortpproxy:yes" in
>>>>>>>
>>> different place
>>>
>>>>>>> in the SDP packet: NOT at the end of the SDP body. The
>>>>>>>
>>> "a=nortpproxy:yes"
>>>
>>>>>>> should be either put at the global attribute level or
>>>>>>>
>>> right below
>>>
>>>>>>> a m= line.
>>>>>>>
>>>>>>> This approach would solve such issue and would be much more
>>>>>>> adequate to the requirement: currently it's not very
>>>>>>>
>>> possible to
>>>
>>>>>>> relay an audio stream and not relay a video stream with the
>>>>>>> current approach.
>>>>>>>
>>>>>>> tks,
>>>>>>> Aymeric MOIZARD / ANTISIP
>>>>>>> amsip - http://www.antisip.com
>>>>>>> osip2 - http://www.osip.org
>>>>>>> eXosip2 - http://savannah.nongnu.org/projects/exosip/
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Users mailing list
>>>>>>> Users at lists.kamailio.org <mailto:Users at lists.kamailio.org>
>>>>>>> http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
>>>>>>>
>>>>>>>
>>>>>> _______________________________________________
>>>>>> Users mailing list
>>>>>> Users at lists.kamailio.org <mailto:Users at lists.kamailio.org>
>>>>>> http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>> *Jérôme Martin **| **LongPhone*
>>>> *Responsable Architecture Réseau*
>>>> 122, rue la Boetie | 75008 Paris
>>>> Tel : +33 (0)1 56 26 28 44
>>>> Fax : +33 (0)1 56 26 28 45
>>>> Mail : *jmartin*@longphone.fr
>>>> Web : www.longphone.com <http://www.longphone.com>
>>>>
>>>>
>>>>
>>>>
>> ----------------------------------------------------------------------
>>
>>>> --
>>>>
>>>> _______________________________________________
>>>> Users mailing list
>>>> Users at lists.kamailio.org
>>>> http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
>>>>
>>>>
>>> --
>>> Daniel-Constantin Mierla
>>> http://www.asipto.com
>>>
>>>
>>> _______________________________________________
>>> Users mailing list
>>> Users at lists.kamailio.org
>>> http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
>>>
>>>
>
> _______________________________________________
> Users mailing list
> Users at lists.kamailio.org
> http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
>
>
--
Daniel-Constantin Mierla
http://www.asipto.com
More information about the sr-users
mailing list