<div dir="ltr">Camille, thanks for the feedback.  Those are great points, and I'll try to complete work on those configuration features soon.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 4, 2016 at 5:33 AM, Camille Oudot <span dir="ltr"><<a href="mailto:notifications@github.com" target="_blank">notifications@github.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p>Hi Armen,<br>
thanks for your patch, I think this feature will be very useful.<br>
I have one comment: if the tcpops module is loaded but no <code>tcp:closed</code> route is defined, a debug log line will complain about it. I think it would make sense to add a module parameter to globally enable or disable the event route lookup and execution.<br>
Besides, as many of the tcpops function can act on a per-socket basis, it would be nice to have this event route called or not called depending on the socket (e.g. using a <code>tcp_connection</code> flag and defining a new tcpops function to set/reset this flag).<br>
What do you think?</p><div class="HOEnZb"><div class="h5">

<p style="font-size:small;color:#666">—<br>Reply to this email directly or <a href="https://github.com/kamailio/kamailio/pull/454#issuecomment-168677156" target="_blank">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZWlhqzzU_0tJOtRyJbcxxQho0d7Iks5pWmxCgaJpZM4G64Qp.gif" width="1"></p>
<div>
<div>
  
  
</div>

</div>
</div></div><br>_______________________________________________<br>
sr-dev mailing list<br>
<a href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a><br>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
<br></blockquote></div><br></div>