httpapitest Small cosmetic fixes
authorOlle E. Johansson <oej@edvina.net>
Fri, 3 Jun 2016 20:25:39 +0000 (22:25 +0200)
committerOlle E. Johansson <oej@edvina.net>
Tue, 7 Jun 2016 06:37:50 +0000 (08:37 +0200)
test/mod_httpapitest/httpapitest.c
test/mod_httpapitest/test/curlapi.cfg

index be50d51..921074d 100644 (file)
@@ -56,6 +56,7 @@ static httpc_api_t httpapi;
 
 /* Exported functions */
 static cmd_export_t cmds[] = {
+       /* Test_http_connect(connection, <URL>, <result pvar>)  - HTTP GET */
        {"test_http_connect", (cmd_function)w_testcurl_connect, 3, fixup_testcurl_connect,
                fixup_free_testcurl_connect,
                REQUEST_ROUTE|ONREPLY_ROUTE|FAILURE_ROUTE|BRANCH_ROUTE},
@@ -123,20 +124,24 @@ static void destroy(void)
 
 
 /*
- * Fix curl_connect params: connection(string/pvar) url (string that may contain pvars) and
- * result (writable pvar).
+ * Fix test_curl_connect params: 
+ * 1. connection(string/pvar) 
+ * 2. url (string that may contain pvars) and
+ * 3. result (writable pvar).
  */
 static int fixup_testcurl_connect(void** param, int param_no)
 {
 
+       /* 1. Connection */
        if (param_no == 1) {
                /* We want char * strings */
                return 0;
        }
-       /* URL and data may contain pvar */
+       /* 2. URL and data may contain pvar */
        if (param_no == 2) {
                return fixup_spve_null(param, 1);
        }
+       /* 3. PVAR for result */
        if (param_no == 3) {
                if (fixup_pvar_null(param, 1) != 0) {
                        LM_ERR("failed to fixup result pvar\n");
index 4cd6ff0..a6d5a8a 100644 (file)
@@ -5,7 +5,7 @@
 #
 #
 
-debug=3
+debug=2
 log_stderror=yes
 fork=no
 
@@ -55,12 +55,12 @@ onsend_route
 # This route is executed at Kamailio start
 event_route[htable:mod-init]
 {
-       xlog("L_ERR", "### Kamailio starting $timef(HH:mm) \n");
+       xlog("L_ERR", "### Kamailio starting $timef(%HH:%mm) \n");
        $var(html) = "";
 
        $var(res) = test_http_connect("lisa", "", "$avp(html)");
 
-       xlog("L_ERR", "-- Lisa http_client connection: $avp(gurka) Result $var(res)\n");
+       xlog("L_ERR", "-- Lisa http_client connection: Result $var(res) Response:\n--------\n$avp(html)\n------\n");
 
 
 }