[Kamailio-Devel] [ openser-Patches-1741375 ] uac_redirect - bflags - Parameter

[SourceForge.net]

Patches item #1741375, was opened at 2007-06-22 09:22
Message generated for change (Settings changed) made by carstenbock
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1741375&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: Accepted
Priority: 5
Private: No
Submitted By: Carsten Bock (carstenbock)
Assigned to: Nobody/Anonymous (nobody)
Summary: uac_redirect - bflags - Parameter

Initial Comment:
Hi,

a small improvement for the uac_redirect-Module:
I have added a parameter called bflags, which allows you to set a branch-flag foir the new branches created by get_redirects().
You can create a branch_route now and implement a special handling for branches added through "get_redirects".
The Patch is against OpenSER Trunk.

Carsten

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

Comment By: Nobody/Anonymous (nobody)
Date: 2008-09-02 08:06

Message:
Logged In: NO 

carten,

README text could say that the flags are set for all new branches that
result from get_redirects call.

on the other hand, would it be more flexible, if the flags that will get
set could be given as an optional argument to get_redirects call instead of
module parameter?  then you could tell a difference, for example, if 3xx
came from pstn or a user of the proxy.

anyway, feel free to commit the patch.

-- juha


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

Comment By: Carsten Bock (carstenbock)
Date: 2008-09-01 11:00

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

By the way, attached an updated patch for the 1.4 tree.
File Added: uac_redirect.patch

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

Comment By: Carsten Bock (carstenbock)
Date: 2008-09-01 10:58

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

Hi Henning,

i thought about commiting this myself, since at least i still need this
feature...
I did, so far, not investigate whom to contact about the patch. Is Bogdan
really still the maintainer of this module?

Carsten

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

Comment By: Henning Westerholt (henningw)
Date: 2008-08-29 16:28

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

Hi Carsten,

what is the status of this patch? Do you still want to have this in the
trunk? Or is this now perhaps not necessary anymore?

Cheers,

Henning

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

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