On Wed, 18 Sep 2013 13:41:34 +0000 Andrew Mortensen admorten@isc.upenn.edu wrote:
On Sep 18, 2013, at 3:54 AM, Timo Teräs timo.teras@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