Thanks for the feedback Daniel.<br>
I just pushed 2 commits from Camille to implement pv module api and use it<br>
in the new module.<br>
I'll also add a note on the documentation about the conflict with the curl<br>
module.<br>
About the name, http_client could be a good one, the current one having<br>
been chosen mainly to highlight the asynchronous processing.<br>
<br>
<br>
On Wed, Jan 20, 2016 at 1:22 PM, Daniel-Constantin Mierla <<br>
notifications@github.com> wrote:<br>
<br>
> I am ok with many modules targeting to offer similar functionality, if<br>
> they have different approach -- like lcr can be achieved with couple of<br>
> modules. If there is a conflict with another module, that needs to be<br>
> documented.<br>
><br>
> Some remarks:<br>
><br>
>    -<br>
><br>
>    using functions from pv module must be done via inter-module api<br>
>    structure. The old method to export via module structure for config<br>
>    functions is prone to issues when prototypes are changed, because the cast<br>
>    will hide that. Many modules export now internal API via a bind structure,<br>
>    en example is to look at sl_load_api() in modules/sl/sl.h and how it is<br>
>    used from other modules, like registrar.<br>
>    -<br>
><br>
>    more like personal opinion, I find the name a bit restrictive for the<br>
>    future, just in case one will want to add some non-async features to it.<br>
>    Maybe it would be better for long term to use something more generic, e.g.,<br>
>    httpc, http_client, ...<br>
><br>
> —<br>
> Reply to this email directly or view it on GitHub<br>
> <https://github.com/kamailio/kamailio/pull/480#issuecomment-173187418>.<br>
><br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/kamailio/kamailio/pull/480#issuecomment-173475562">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZcY6YamFqHN6I2dO4IQ08IncCRJNks5pcHYOgaJpZM4HG-ko.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/pull/480#issuecomment-173475562"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>