[sr-dev] git:master:2e567ce1: dispatcher: Adding new feature for counting successful requests before placing destination from inactive to active state (code)

[Daniel-Constantin Mierla]

Thanks for catching it. I pushed a patch to fix it.

On analysis, I discovered that both probing_threshold and
inactive_threshold are defined as cfg framework variables, but actually
the code is using the local vars, thus changing via kamcmd cfg.seti
dispatcher probing_threshold ... doesn't have effect. I will follow up
with a fix on that in the near future.

Cheers,
Daniel

On 13/01/15 17:24, Ovidiu Sas wrote:
> This commit breaks the code: undefined symbol: inactive_threshold
>
> On Fri, Jan 9, 2015 at 2:11 PM, alezzandro <alezzandro at gmail.com> wrote:
>> Module: kamailio
>> Branch: master
>> Commit: 2e567ce1baac21d6e063a6924a80aff06da803cf
>> URL: https://github.com/kamailio/kamailio/commit/2e567ce1baac21d6e063a6924a80aff06da803cf
>>
>> Author: alezzandro <alezzandro at gmail.com>
>> Committer: alezzandro <alezzandro at gmail.com>
>> Date: 2014-12-24T00:52:53+01:00
>>
>> dispatcher: Adding new feature for counting successful requests before placing destination from inactive to active state (code)
>>
>> ---
>>
>> Modified: modules/dispatcher/config.c
>> Modified: modules/dispatcher/config.h
>> Modified: modules/dispatcher/dispatch.c
>> Modified: modules/dispatcher/dispatch.h
>> Modified: modules/dispatcher/dispatcher.c
>>
>> ---
>>
>> Diff:  https://github.com/kamailio/kamailio/commit/2e567ce1baac21d6e063a6924a80aff06da803cf.diff
>> Patch: https://github.com/kamailio/kamailio/commit/2e567ce1baac21d6e063a6924a80aff06da803cf.patch
>>
>>
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>
>

-- 
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda