<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14900">View Change</a></p><p>3 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//COMMIT_MSG">Commit Message:</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//COMMIT_MSG@7">Patch Set #2, Line 7:</a> <code style="font-family:monospace,monospace">Mofified funcs/func_curl.c so that 404's are handled as a failure.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This needs to follow the wiki page[1] for commit messages.</p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://wiki.asterisk.org/wiki/display/AST/Commit+Messages</p></li></ul></li><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 style="white-space: pre-wrap; word-wrap: break-word;">Excess space at the end</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 style="white-space: pre-wrap; word-wrap: break-word;">This is a breaking behavior change. Such a behavior change should be optional for users so they aren't caught off guard. This also only applies to 404, I think there should be the ability to actually inspect the response code from the dialplan and take action based on it instead so that specific response code behavior is not hard coded.</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: Mon, 14 Sep 2020 12:56:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>