On a Linux 64 bits Ubuntu, I'm trying to use the http_async_client to send a POST message but whatever I set, it always suspends the current route contrary to the content of the documentation. When I look at the C code it seems to be done on purpose as the return for the POST is different than the return from the get: static int w_http_async_get(sip_msg_t *msg, char *query, char* rt) { ... return async_send_query(msg, &sdata, NULL, act);
}
static int w_http_async_post(sip_msg_t *msg, char *query, char* post, char* rt) { ... if(async_send_query(msg, &sdata, &post_data, act)<0) return -1;
/* force exit in config */ return 0; }
I was expecting the post function to work as the get one and to finish with something like: return async_send_query(msg, &sdata, &post_data, act);
Based on the document it looks like a bug but the comment bothers me and let me think it might be done on purpose (which one?)
Definitely a bug, thanks for reporting! I've just pushed a fix on master. If you could try and give feedback I'll backport then.
On Fri, Aug 12, 2016 at 2:25 PM, AlainDubois78 notifications@github.com wrote:
On a Linux 64 bits Ubuntu, I'm trying to use the http_async_client to send a POST message but whatever I set, it always suspends the current route contrary to the content of the documentation. When I look at the C code it seems to be done on purpose as the return for the POST is different than the return from the get: static int w_http_async_get(sip_msg_t *msg, char *query, char* rt) { ... return async_send_query(msg, &sdata, NULL, act);
}
static int w_http_async_post(sip_msg_t *msg, char *query, char* post, char* rt) { ... if(async_send_query(msg, &sdata, &post_data, act)<0) return -1;
/* force exit in config */ return 0;
}
I was expecting the post function to work as the get one and to finish with something like: return async_send_query(msg, &sdata, &post_data, act);
Based on the document it looks like a bug but the comment bothers me and let me think it might be done on purpose (which one?)
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kamailio/kamailio/issues/744, or mute the thread https://github.com/notifications/unsubscribe-auth/AF36ZdU-9YfQ27wpMr9DXr9cYDUmmq93ks5qfGZNgaJpZM4JjCSr .
sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
I actually already tested as I made the proposed modification on my own PC. I works just as expected Thanks for the quick feedback
Ok, thank you, going to backport.
On Fri, Aug 12, 2016 at 2:51 PM, AlainDubois78 notifications@github.com wrote:
I actually already tested as I made the proposed modification on my own PC. I works just as expected Thanks for the quick feedback
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kamailio/kamailio/issues/744#issuecomment-239437695, or mute the thread https://github.com/notifications/unsubscribe-auth/ADip4zFibiG28VfzpaGpfxk8eVGqWSS4ks5qfGw5gaJpZM4JjCSr .
Closed #744.
-
AlainDubois78 -
Federico Cabiddu -
Federico Cabiddu