custom timeout for http CIDlookup

ops messed the heading

custom timeout for http CIDlookup

Changing to feature request.

Unfortunately there are no options to CURL to provide this, which is how it is currently implemented. It will require a change in implementation, probably using an AGI script, in order to add a timeout.

You could supply your own local version of an http script, and then have that script call the real http service and apply a timeout as requested. Here's something from a quick google search that may help get started (still requires some level of understanding programming a php based script):

http://blog.unitedheroes.net/curl/

thanks for your advice

after doing a small search on google and tried this which works

right under "case "http":" in the cidlookup module I added

$cl = curl_init();
curl_setopt($cl, CURLOPT_CONNECTTIMEOUT, 5);

it worked as expected

I am confused. Are you saying you added those 2 lines to the functions.inc.php file and it made the CURL operation in Asterisk timeout? That doesn't make sense, the CURL function in Asterisk and the CURL functions in PHP don't have anyhing to do do with each other?

well i'm not a programmer but i tried playing around with what sounded like common sense to me and it worked i have a lookup to a remote server which host our crm data and whenever that site went down (not often) i had no incomming calls they would all get stuck waiting for a response, i inserted these two lines and disconnected the internet, the calls came thru but of course without the name

i think it probbally initilizes the curl command globally and might need to be cleaned up afterwords

by the way, you can try this (asterisk modification):

--- func_curl.c 2007-10-26 06:54:30.000000000 -0700
+++ func_curl.modified.c        2008-06-24 07:17:37.000000000 -0700
@@ -104,7 +104,7 @@
                if (!(*curl = curl_easy_init()))
                        return -1;
                curl_easy_setopt(*curl, CURLOPT_NOSIGNAL, 1);
-               curl_easy_setopt(*curl, CURLOPT_TIMEOUT, 180);
+               curl_easy_setopt(*curl, CURLOPT_TIMEOUT, 5);
                curl_easy_setopt(*curl, CURLOPT_WRITEFUNCTION, WriteMemoryCallback);
                curl_easy_setopt(*curl, CURLOPT_USERAGENT, global_useragent);
        }

btw - forgot to request, can you run some tests on your change. Can you try setting it to 0 and then a couple of other times, and run some tests each time to time it and see if the timeout is really correlating with those times you set? (0 should be no timeout).

well seems like i was too fast posting that as it stopped working again now i have to figure out what did help and now stopped i'll keep you updated

If you are compiling Asterisk from source, the patch I supplied should be very risk free, you just need to decide what timeout you want to set - my example uses 5 seconds. The file is in the asterisk source tree: ~funcs/func_curl.c

foxed as part of #4823