<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7835">View Change</a></p><p>Patch set 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Giving yourself a +1. Mhm. Interesting.<br>Corey was about creating a separate commit, so he is able to see it via the headline already. </p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyway, complete different approach: What about keeping the sRTP part and remove sRTP in each package list? Rationale: All supported platforms come with an sRTP library. However, none of the platforms compiled it with OpenSSL enabled. Therefore AES-GCM is not possible – supported in Asterisk 15. And more interesting for Asterisk as a VoIP server: AES-IN are not used. Finally, a security library should come/use the latest security patches. Only a few of the supported platforms offer the latest version. Even then, it is not 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 platform but built manually, especially for a server like Asterisk. Therefore: What about keeping the checkout of libSRTP and removing the install-package-requests at the top?</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 08:08:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>