Re: [sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-10-07 Thread Stefan Mititelu
@gaaf there are actually 2 conditions that consider breaking out of reconnect loop on success: 1'st one, on success, will succeed opening channel, thus will never go further: ``` // open channel - failed if (log_ret != AMQP_RESPONSE_NORMAL) { ... } ``` 2'nd one: ``` // reconnect if (reconnect_at

Re: [sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-09-19 Thread Stefan Mititelu
Added MIT License for dev files in commit 3c9f88b96849e54af1559a85d61d04e2ac00cd4c. -- 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/786#issuecomment-247990646

Re: [sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-09-19 Thread Stefan Mititelu
Merged #786. -- 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/786#event-793991805___ sr-dev mailing list sr-dev@lists.sip-router.org http://

Re: [sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-09-19 Thread ng-voice GmbH
Just a quick note: This Rabbit-MQ module has a different approach, than the Kazoo Modules. The intention is to make it generally available for a Rabbit-MQ infrastructure, rather than Kazoo specific. I wrote the module together with Stefan. -- You are receiving this because you are subscribed to

Re: [sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-09-19 Thread Daniel-Constantin Mierla
It can be merged. Then add the copyright header to the files that don't have one. Some (I guess imported from external projects) have MIT and that is compatible with gpl, so it's ok. For those that don't have, you can choose your preference between GPL, BSD (both accepted so far in the project)

[sr-dev] [kamailio/kamailio] rabbitmq: Add new module (#786)

2016-09-19 Thread Stefan Mititelu
Add new rabbitmq module. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/786 -- Commit Summary -- * rabbitmq: Add new module -- File Changes -- A modules/rabbitmq/Makefile (14) A modules/rabbitmq/README (244) A modules/r