Re: [sr-dev] [kamailio/kamailio] module http_client - support for keeping connections open (#542)

2016-04-20 Thread filiplx
I've tried to use the modparam "keep_connections", and it works! But like the other default parameters, it must be specified _before_ the connection definition to have effect, which I still find unintuitive. It would be neat if this was documented. --- You are receiving this because you are

Re: [sr-dev] [kamailio/kamailio] module http_client - support for preemptive authentication (#515)

2016-03-29 Thread filiplx
This works for me when I tested using modparams. Thank you very much! --- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] module http_client - support setting default http proxy (#541)

2016-03-29 Thread filiplx
I used http_connect(). When I changed the order of the modparams so that the default proxy config comes before the connection definitions, it worked. E.g.: # this works! modparam("http_client", "httpproxy", "proxyhost") modparam("http_client", "httpproxyport", 8181) modparam("http_client",

Re: [sr-dev] [kamailio] module http_client - support setting default http proxy (#541)

2016-03-24 Thread filiplx
Did not manage to get default proxy to work. I used the same config as I used with my home-brewed version of http_client. The API calls work, it's just the proxy config that isn't applied. # # Config # #!define API_SERVER_A "apiA=>http://name:p...@api01.host.com; #!define API_SERVER_B

Re: [sr-dev] [kamailio] module http_client - support setting default http proxy (#541)

2016-03-19 Thread filiplx
I will try to test it today. --- 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/541#issuecomment-198266536___ sr-dev mailing list

Re: [sr-dev] [kamailio] module http_client - support for keeping connections open (#542)

2016-03-15 Thread filiplx
According to this: https://curl.haxx.se/libcurl/c/CURLOPT_MAXCONNECTS.html , libcurl can per default keep up to 5 open persistent connections in the handle. So as long as 5 or fewer connections are used, everything is fine. Perhaps there should be a module option to set this to a higher value?

[sr-dev] [kamailio] module http_client - support for keeping connections open (#542)

2016-03-11 Thread filiplx
The current implementation uses curl_easy_init() to create a CURL easy handle and curl_easy_cleanup() to destroy the handle, including all open connections within it. If curl_easy_cleanup() isn't called, it's possible to reuse an open connection for the next http call. To reset the CURL

[sr-dev] [kamailio] module http_client - support setting default http proxy (#541)

2016-03-11 Thread filiplx
The code on master branch is _almost_ done. What's missing is this: /** * functions.c * curL_query_url() */ if (default_http_proxy != NULL) { res |= curl_easy_setopt(curl, CURLOPT_PROXY, default_http_proxy); } if (default_http_proxy_port > 0) { res |=

[sr-dev] [kamailio] module http_client - support for preemptive authentication (#515)

2016-02-19 Thread filiplx
It would be nice if it was possible to configure the module to use preemptive authentication. I forked the module to add it myself in a primitive way since I needed it only for Basic Authentication, but it would be better to have it configurable. In functions.h // Before: res |=