[sr-dev] [kamailio/kamailio] evapi: Added destination address on evapi_relay (#596)

Eloy Coto notifications at github.com
Wed May 4 12:58:16 CEST 2016


Hey @miconda 

About the memory leak, I think that now is fixed but if you can confirm I'll be great. 
About indention, should be ok now. Sorry!

About tags/IP:PORT. When I did this I thought about tagging, but after a few changes I used HOST:PORT. We use evapi mainly with Cgrates, but in the distributed environment when we have n instances of Cgrates we need to send the data to each cgrates, if we send to all instances will be billed few times. 

The idea to use host:port is mainly about host and port is given in the `evapi:connection-new`, so the info can be saved in htables + can be compared with the service discovery tool (in our case consul), so I think that is the easiest way. 

If we use tag (uuid for example) more code will be needed in kamailio.cfg, so the simple way that I  found is using host+port. I spoke yesterday with @danbogos about this, and he was agree with my use case.

If you need I can squash the commits into one, the second one doesn't add a real value. 
Regards






---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/596#issuecomment-216828268
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160504/fe137a86/attachment.html>


More information about the sr-dev mailing list