<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">Not quite sure about this change.  ast_tcptls_client_create() and ast_tcptls_client_start() are public APIs and we're changing their behavior.  app_externalivr and res_http_websocket don't appear to do their own closes on failures in client_create() of client_start().</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The tcptls.c code will close the fd before the iostream is created and the iostream code will close the fd after the iostream is created. Feel free to test for yourself.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I see that for websocket  now but humor me and check app_externalivr.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15024">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15024">change 15024</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/+/15024"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ib556b0a0c95cca939e956886214ec8d828d89606 </div>
<div style="display:none"> Gerrit-Change-Number: 15024 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 02 Oct 2020 18:08:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>