[sr-dev] acc, dialog: dialog duration for timed-out dialogs change

[Daniel-Constantin Mierla]

Hello,

if I understood right, you want the calls that are cut after 30sec (or
so) because the ACK was not routed to be stored with duration 0, right?
If yes, then it should be made configurable via module parameter, with
default of what was so far, because those calls also cost money in most
of the cases, being answered.

As an extra note, I don't see a reason declare and to give the old state
as a string, should be given as the associated integer ID (e.g.,
DLG_STATE_CONFIRMED) and compare params->dlg_data casting to unsigned int.

Cheers,
Daniel

On 16/04/15 11:51, Lucian Balaceanu wrote:
> Hi,
>
> I would like to propose a patch in which the dialog duration displayed
> in the cdrs is:
>     /0 -> timed-out not acknowledged dialogs
>     duration from creation to timeout -> timed-out acknowledged dialogs
> /
> I am still testing the solution for the master branch while tests pass
> on a previous version.
> Any comments on the code? (I am not particularly proud of the string
> defines in the dlg_cb.h, but am unsure where to include some const
> variable accessible to both acc and dialog to pass as calback param
> for dialog expiration)
>
> Thank you,
> Lucian Balaceanu
>
>
> _______________________________________________
> 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
Kamailio World Conference, May 27-29, 2015
Berlin, Germany - http://www.kamailioworld.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20150416/2fb5c44d/attachment.html>