<p><a href="https://gerrit.asterisk.org/c/asterisk/+/14900">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14900/2/funcs/func_curl.c">File funcs/func_curl.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/14900/2/funcs/func_curl.c@659">Patch Set #2, Line 659:</a> <code style="font-family:monospace,monospace">     long http_code = 0;     </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Excess space at the end</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack. This will be corrected.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/14900/2/funcs/func_curl.c@745">Patch Set #2, Line 745:</a> <code style="font-family:monospace,monospace">    if (http_code == 404){</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This is a breaking behavior change. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">For the errors perhaps we can have a configuration that would specify which setting with which errors are considered a failure? If that is sufficient where would the configuration be? asterisk.conf or somewhere else?</p><p style="white-space: pre-wrap; word-wrap: break-word;">This fix was done for Asterisk Real time where there is no place to check for errors. Either it passed or it failed. If it passed then we don't move over to the next real time option if not then we do.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14900">change 14900</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/14900"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ib10e14ce240e8ceb659da63ea11b1d91ce6b179a </div>
<div style="display:none"> Gerrit-Change-Number: 14900 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Dovid Bender <dovid@telecurve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 27 Sep 2020 13:13:43 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>