<p>Kevin Harwell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7835">View Change</a></p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ddd;">-Code-Review</span></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Giving yourself a +1. Mhm. Interesting.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hey if you can give yourself a -1 why not a +1 :-D Actually all I had done was update the commit message with why sRTP was being removed and was giving a +1 to the overall patch that you uploaded.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Corey was about creating a separate commit, so he is able to see it<br>via the headline already.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyway, complete different approach: What about keeping the sRTP<br>part and remove sRTP in each package list? Rationale: All supported<br>platforms come with an sRTP library. However, none of the platforms<br>compiled it with OpenSSL enabled. Therefore AES-GCM is not possible<br>– supported in Asterisk 15. And more interesting for Asterisk as a<br>VoIP server: AES-IN are not used. Finally, a security library<br>should come/use the latest security patches. Only a few of the<br>supported platforms offer the latest version. Even then, it is not<br>updated when an issue is found.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In other words: sRTP should not rely on the package from the<br>platform but built manually, especially for a server like Asterisk.<br>Therefore: What about keeping the checkout of libSRTP and removing<br>the install-package-requests at the top?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I wouldn't be opposed to this, and based on your rationale and what the wiki pages says I think I prefer this too. I'll go ahead and remove my +1 for this so it doesn't accidentally get committed before more people can offer their opinions.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7835">change 7835</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/7835"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Icbf4b562f81f7c05bd24a3805bd46c0beb4ebd44 </div>
<div style="display:none"> Gerrit-Change-Number: 7835 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 19 Jan 2018 15:58:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>