On Wed, 18 Sep 2013 13:41:34 +0000
Andrew Mortensen <admorten(a)isc.upenn.edu> wrote:
On Sep 18, 2013, at 3:54 AM, Timo Teräs <timo.teras(a)iki.fi>
wrote:
sca_call_info_uri_update() return value is passed
directly as
return value of script function "sca_call_info_update". Return
value 0 means 'exit script' but that is unwanted. Fix it to
just continue with script execution normally.
---
Andrew, does this look ok? Can I push this, or will you do it?
Should go to master and 4.0 branch at least.
Looks good, thanks for finding it. I have a number of changes
committed to the admorten/sca branch, so if you don't mind I'll add
the fix there, and cherry pick it with the others into the 4.x and
master branches.
Thanks again!
Ok, great.
Relatedly, I found another issue, but am not sure what is really the
right fix.
It seems that even with Call-Info header, the AoR is checked against
From/To headers.
I have setup where the SCA number is routed via ENUM. So I have
remote-box doing ENUM lookup, and sending to ENUM number so the From
field is caller-id, and To is based on the ENUM number. Only R-URI and
Contact header are converted and canonicalized to the format of AoR.
So I'm wondering is it correct to use only From/To for AoR checks, or
should also R-URI and/or Contact be checked instead or in addition.
Alternative is that I need to somewhere always overwrite the To field
in kamailio script which sounds hacky.
I tried to read the specs, but didn't find any exact wording to this.
Any suggestions?
- Timo