<p>This latest commit implements some of the TODO's in the curl module.</p>

<ul>
<li>default client_cert, client_key, cacert and verifyserver modparams implemented</li>
<li>added a cipher_suites modparam so the default list provided by libcurl can be overridden (e.g. enabling elliptic-curve ciphers)</li>
<li>added per-connection value for verifyserver</li>
</ul>

<p>Here are some followup questions for <a href="https://github.com/oej" class="user-mention">@oej</a> or anyone else who may use the curl module.<br>
Should the <a href="http://curl.haxx.se/libcurl/c/CURLOPT_SSL_VERIFYHOST.html">verify_host</a> parameter be configurable as well as verify_peer?<br>
Is the intention to make the client cert/key configurable per-connection?</p>

<p>If we add more parameters to the curlcon modparam, it will easily get unmanageable.<br>
I suggest two modparams, curlcon and curlcon_param. The first sets up the url and parameters (this is unchanged). The second will add parameters to a curlcon that is already defined. E.g.</p>

<pre><code>modparam("curl", "curlcon", "apione=>https://myserver.example.com/url;timeout=12")
modparam("curl", "curlcon_param", "apione=>verifyserver=1")
modparam("curl", "curlcon_param", "apione=>clientkey=/var/certs/myserver.example.com.pem")
</code></pre>

<p>Any thoughts? Have I missed anything else in this area?</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/kamailio/kamailio/pull/481'>https://github.com/kamailio/kamailio/pull/481</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>curl: Add client key/certificate to curl_connect</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/481/files#diff-0">modules/curl/curl.c</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/481/files#diff-1">modules/curl/curl.h</a>
    (6)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/481/files#diff-2">modules/curl/curlcon.c</a>
    (24)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/481/files#diff-3">modules/curl/doc/curl_admin.xml</a>
    (26)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/481/files#diff-4">modules/curl/functions.c</a>
    (113)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/kamailio/kamailio/pull/481.patch'>https://github.com/kamailio/kamailio/pull/481.patch</a></li>
  <li><a href='https://github.com/kamailio/kamailio/pull/481.diff'>https://github.com/kamailio/kamailio/pull/481.diff</a></li>
</ul>

<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/481">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZVD0DST6X6o3jH0BS_5F02wzAqfIks5pbn3TgaJpZM4HH-3o.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/481"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>