The connection timeout was hardcoded to 1 sec. This can now be set from cfg file.
If the redis server became unresponsive kamailio would wait for the standard Linux TCP timeout for an answer (very long time).
This patch has added an timeout value for redis commands.
The default timeout is 1 sec. [30-redis-timeout.txt](https://github.com/kamailio/kamailio/files/198503/30-redis-timeout.txt)
--- 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/issues/557
We have found a few issues with this patch. I will send a new patch later.
--- 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/issues/557#issuecomment-204278542
It is better to make a pull request, in that way the patch can be reviewed directly on github and it is compiled to see if it rises any issues. After that, merging the patch is just one click.
--- 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/issues/557#issuecomment-204283283
Hi Daniel,
OK. I will try to do that with the next patch.
/Morten
--- 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/issues/557#issuecomment-204309111
Pull request #559 created.
--- 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/issues/557#issuecomment-204350390
Closed #557.
--- 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/issues/557#event-611173676
Closing this one an continuing on the pull request if there is anything else to be discussed.
--- 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/issues/557#issuecomment-204368623
-
Daniel-Constantin Mierla -
misaksen