<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8906">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;">I'm not sure that this is adequate enough of a solution. We have a<br>hard-coded maximum of 64 in ast_sip_transport_state.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Very possible I'm misunderstanding, but I think the hard-coded maximum of 64 limits the number of ciphers which can be configured per transport, where PJ_SSL_SOCK_MAX_CIPHERS defines the maximum number of ciphers that can exist on the system?  So with this patch you're still limited to configuring up to 64 ciphers per transport, but those 64 can be chosen from the full list of ciphers that exist instead of just the first 100.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Obviously fixing the 64 would be an ABI change so I'll leave it to others to comment on if it can be fixed in 13/15.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8906">change 8906</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/8906"/><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: Iea620f03915a1b873e79743154255c3148a514e7 </div>
<div style="display:none"> Gerrit-Change-Number: 8906 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 02 May 2018 13:54:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>