[OpenSER-Devel] [ openser-Bugs-1986176 ] cr_route does not accept any pseudo-variable

[SourceForge.net]

Bugs item #1986176, was opened at 2008-06-06 09:03
Message generated for change (Comment added) made by carstenbock
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1986176&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: None
Priority: 5
Private: No
Submitted By: Carsten Bock (carstenbock)
Assigned to: Henning Westerholt (henningw)
Summary: cr_route does not accept any pseudo-variable

Initial Comment:
Hi,

According to the documentation, cr_route should accept any pseudo-variable as parameter as carrier.

"carrier - The routing tree to be used. Additional to a string any pseudo-variable could be used as input."
(http://openser.org/docs/modules/devel/carrierroute.html#id2533392)

However, the function currently only accepts AVP's and not any PVar (we have only tested the "$hdr(X-Carrier)"). As soon as we do

$avp(i:100) = $hdr(X-Carrier);

and then

cr_route("$avp(i:100)", ...

it works, but not using directly

cr_route("$hdr(X-Carrier)", ...

Carsten

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

>Comment By: Carsten Bock (carstenbock)
Date: 2008-06-09 09:22

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

Hi,

We did some testing and it seems to work fine now, thanks!

Carsten

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

Comment By: Henning Westerholt (henningw)
Date: 2008-06-06 16:58

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

Hi Carsten,

i've added some code to support this. I've did some quick tests, it seems
to work. Would be great if you could also do some test, then this could be
closed.

Henning

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

Comment By: Henning Westerholt (henningw)
Date: 2008-06-06 15:45

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

Hi Carsten,

you're correct. This is in fact a regression from 1.3, in
cr_tree_rewrite_uri it was possible to use any PV. I think this was lost
during the porting. I'll investigate, and fix this.

Thanks for the report for now,

Henning

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

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