<p>Sean Bright <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;"><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<br>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<br>maximum of 64 limits the number of ciphers which can be configured<br>per transport, where PJ_SSL_SOCK_MAX_CIPHERS defines the maximum<br>number of ciphers that can exist on the system?  So with this patch<br>you're still limited to configuring up to 64 ciphers per transport,<br>but those 64 can be chosen from the full list of ciphers that exist<br>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<br>others to comment on if it can be fixed in 13/15.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, I think you're right. They are two separate issues so I think this patch is good-ish as is.</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 14:00:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>