<p>George Joseph <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/5877">View Change</a></p><p>Patch set 3:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)<br>> What's the use case?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This is mainly meant for the administrative interfaces: AMI, CLI,<br>and HTTP for rest api.  Obviously it would be up to a local admin<br>to decide if they wanted any of the individual components to cause<br>Asterisk startup.  For my own use case I intend to enable AMI only<br>so the unprivileged service running on localhost can force Asterisk<br>to start.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Ah, gotcha.  I'd add the "start via ami or ari" case to the readme.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>As for the sip-tcp / sip-tls listeners, I'm not sure of a<br>reasonable use case for enabling socket activation on those.  I<br>documented them since they use ast_tcptls_server_start and thus can<br>use socket activation.  If wanted I can remove the socket files for<br>chan_sip to show that they are not the intended use case?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'd remove the sip references.  I can't think of a use case where someone would want asterisk to automatically start on a sip tcp or tls attempt.  Might be a security risk.</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/5877/3/contrib/systemd/asterisk-http.socket">File contrib/systemd/asterisk-http.socket:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/5877/3/contrib/systemd/asterisk-http.socket@1">Patch Set #3, Line 1:</a> <code style="font-family:monospace,monospace">[Unit]</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">This was the first one I tested, it worked for me.  Can you<br>double-check your settings between asterisk-http.socket and<br>asterisk/http.conf?  The bind address needs to be an exact match.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yeah, asterisk was listening on '[::]' but the unit was listening on '0.0.0.0'</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/5877/3/contrib/systemd/asterisk-http.socket@1">Patch Set #3, Line 1:</a> <code style="font-family:monospace,monospace">[Unit]</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">http doesn't seem to work.  asterisk starts but the socket doesn't<br>seem to be passed through to asterisk.</p></blockquote></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/5877">change 5877</a>. To unsubscribe, 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/5877"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Id814ee6a892f4b80d018365c8ad8d89063474f4d </div>
<div style="display:none"> Gerrit-Change-Number: 5877 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 19 Jun 2017 15:49:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>