[OpenSER-Devel] [ openser-Bugs-1815162 ] force_rtp_proxy("l") in "on_reply_route" doesn't work

[SourceForge.net]

Bugs item #1815162, was opened at 2007-10-17 17:54
Message generated for change (Settings changed) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1815162&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver devel
>Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
Assigned to: Bogdan-Andrei Iancu (bogdan_iancu)
Summary: force_rtp_proxy("l") in "on_reply_route" doesn't work

Initial Comment:
Hi, the doc about nathelper module appears:

------------------------------------------------------
l - force "lookup", that is, only rewrite SDP when corresponding session is already exists in the RTP proxy. Only makes sense for SIP requests, replies are always processed in "lookup" mode.l - force "lookup", that is, only rewrite SDP when corresponding session is already exists in the RTP proxy. Only makes sense for SIP requests, replies are always processed in "lookup" mode.
------------------------------------------------------

So in replies it asumes always "lookup" mode, but the fact is that if you set in "on_reply_route":

  force_rtp_proxy("l");

then it doesn't work and is mandatory to set:

  force_rtp_proxy();

IMHO this function should let the user to specific "lookup" mode in replies even if it's not necesary, and it should work.

Best regards.

----------------------------------------------------------------------

Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2007-11-13 19:20

Message:
Logged In: YES 
user_id=1275325
Originator: NO

Hi,

I made a fix on trunk (1.3) - could you confirm it works ok now? Then I
will do the backport.

Thanks and regards,
Bogdan

----------------------------------------------------------------------

Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2007-11-13 10:35

Message:
Logged In: YES 
user_id=1275325
Originator: NO

I will invetigate on this.

regards,
bogdan

----------------------------------------------------------------------

Comment By: Iñaki Baz (ibc_sf)
Date: 2007-10-17 17:56

Message:
Logged In: YES 
user_id=1844020
Originator: YES

Or at least document in the doc that it's mandatory in replies to call the
funciont with no parameter.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1815162&group_id=139143