On 4/8/10 5:16 PM, Alex Balashov wrote:
I noticed this too.
On 04/08/2010 11:11 AM, Andreas Granig wrote:
> Hi,
>
> In the docs at
>
http://kamailio.org/docs/modules/3.0.x/modules/tm.html#fr_inv_timer it
> says that fr_inv_timer_avp should be defined like this, without $ or
> $avp:
>
> modparam("tm", "fr_inv_timer_avp", "my_fr_inv_timer")
>
> In kam <= 1.5 I did it like that:
>
> modparam("tm", "fr_inv_timer_avp",
"$avp(s:callee_fr_inv_timer)")
if you set config to kamailio compat mode via #!KAMAILIO then it accepts
the kamailio format where all avps are specified in PV format:
$avp(...). Doc needs some update :-) ...
>
> which doesn't give me an error on startup with kam-3.0 either, but the
> timer doesn't get fired (I use seconds for that as noted in the docs).
Well, seems to be a bug in code, I thought the timer is set in
miliseconds even for avps. It is a incoherence imo, the fr_timer and
fr_inv_timer module parameters are in miliseconds, but when given via
avps expects seconds, making impossible to have dynamic timeouts less
than 1 sec via avp. There is t_set_fr() but would be easier to have all
timeouts using same unit.
When #!KAMAILIO is defined and the value of timeout avp is less than
120, then it is multiplied with 1000 (auto-correction from second to
milisecond), but since it actually second for AVP case, will result in a
very long timeout :-).
I will fix it.
Thanks,
Daniel
If I change it
to
modparam("tm", "fr_inv_timer_avp", "callee_fr_inv_timer")
then I get the error "malformed or non AVP callee_fr_inv_timer AVP
definition", same with setting it to "s:callee_fr_inv_timer".
Anyhow, it seems to be deprecated anyways, so I'm looking to get
t_set_fr() working. I'm just curious how I can use a var or AVP loaded
from DB to set the value on-the-fly? t_set_fr(...) seems to allow only
constants to be set. Couldn't find anything in the docs regarding
that one.
Thanks,
Andreas
_______________________________________________
Kamailio (OpenSER) - Users mailing list
Users(a)lists.kamailio.org
http://lists.kamailio.org/cgi-bin/mailman/listinfo/users
http://lists.openser-project.org/cgi-bin/mailman/listinfo/users
--
Daniel-Constantin Mierla *
http://www.asipto.com/ *
http://twitter.com/miconda *
http://www.linkedin.com/in/danielconstantinmierla