<h3>Description</h3>
<p>ACC module <a href="http://kamailio.org/docs/modules/devel/modules/acc.html#acc.p.cdr_extra" rel="nofollow">cdr_extra modparam</a> allows saving additional custom values in cdr entries using dlg_vars.</p>
<p>When you define a new value to be saved:</p>
<p><code>modparam("acc", "cdr_extra", "c1=$dlg_var(caller)")</code></p>
<p>and you set a $dlg_var(c1) in the logic, everything works as expected.</p>
<p>But if you set it to $null or you don't even define it, it is saved as '' (empty string).</p>
<p>Would be nice that setting to $null would store the underlying null value (NULL in SQL database) and not defining it would lead to not even including it in the final INSERT?</p>
<h3>Troubleshooting</h3>
<h4>Reproduction</h4>
<p>Define a new value to be saved:</p>
<p><code>modparam("acc", "cdr_extra", "c1=$dlg_var(caller)")</code></p>
<p>And in the kamailio.cfg logic:</p>
<ul>
<li>
<p>If you set $dlg_var(c1) to any string ('', 'foo', '2'), it is saved as expected ('', 'foo', '2').</p>
</li>
<li>
<p>If you set $dlg_var(c1) to an integer different from 0 (1, 2, 3), it is saved as a string containing that number ('1', '2', '3'). This seems normal as dlg_vars can only store strings.</p>
</li>
<li>
<p>If you set $dlg_var(c1) to 0, it is saved as an empty string (''). Seems normal too.</p>
</li>
<li>
<p>If you set $dlg_var(c1) to $null, it is saved as an empty string ('').</p>
</li>
<li>
<p>If you don't define $dlg_var(c1) at all, it is saved as an empty string ('').</p>
</li>
</ul>
<h4>Debugging Data</h4>
<p>Both <a href="https://github.com/kamailio/kamailio/blob/e101f21c37238862a474c50218d11a475d121ee0/src/modules/acc/acc_cdr.c#L223"><em>extra2strar</em></a> and <a href="https://github.com/kamailio/kamailio/blob/e101f21c37238862a474c50218d11a475d121ee0/src/modules/acc/acc_cdr.c#L231"><em>extra2strar_dlg_only</em></a> functions defined in <a href="https://github.com/kamailio/kamailio/blob/master/src/modules/acc/acc_extra.c"><em>acc_extra.c</em></a> set <em>cdr_type_array</em> entry to TYPE_NULL, but that array is not used in subsequent <a href="https://github.com/kamailio/kamailio/blob/e101f21c37238862a474c50218d11a475d121ee0/src/modules/acc/acc_cdr.c#L239">for loop</a>.</p>
<h3>Possible Solutions</h3>
<p>Modify the logic in the <a href="https://github.com/kamailio/kamailio/blob/e101f21c37238862a474c50218d11a475d121ee0/src/modules/acc/acc_cdr.c#L239">for loop</a> so that it invokes VAL_NULL macro for null values using <em>cdr_type_array</em>.</p>
<p>With this change both $null values and undefined values would use underlying NULL value. I could try to make further changes to avoid the key in the insert for undefined values instead of setting it to NULL.</p>
<h3>Additional Information</h3>
<ul>
<li><strong>Kamailio Version</strong> - output of <code>kamailio -v</code></li>
</ul>
<pre><code>version: kamailio 4.4.6 (x86_64/linux) 0a8379-dirty
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 0a8379 -dirty
compiled on 12:23:36 Dec 20 2017 with gcc 6.3.0

</code></pre>
<ul>
<li><strong>Operating System</strong>:</li>
</ul>
<pre><code>Linux 4.9.0-3-amd64 #1 SMP Debian 4.9.30-2+deb9u3 (2017-08-06) x86_64 GNU/Linux
</code></pre>
<p>Is there any reason I missed for not doing this change (<a href="https://github.com/kamailio/kamailio/blob/e101f21c37238862a474c50218d11a475d121ee0/src/modules/acc/acc_extra.c#L233">this comment</a> makes me think so)? If not, I can try to provide a PR with these changes.</p>
<p>Regards,</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/issues/1378">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36Zc0SQfMIJFwceRWAllTkOqVV7-SSks5tCPf2gaJpZM4RITCs">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZUpaa9bUMJM_XDETTpdUxZuB73W0ks5tCPf2gaJpZM4RITCs.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/issues/1378"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"acc: set cdr_extra values to NULL (#1378)"}],"action":{"name":"View Issue","url":"https://github.com/kamailio/kamailio/issues/1378"}}}</script>