<div dir="ltr">Mark,<div><br></div><div>From memory SSL_CIPHER_is_AES is a BoringSSL addition isnt it? I did a quick look over the OpenSSL source and it does not seem like its been added either.</div><div><br></div><div>I havent had a chance to compile this yet to confirm it, but if correct then this is not compatible with OpenSSL and possibly other SSL libraries.</div><div><br></div><div>Regards,<br>Mathew</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 28, 2015 at 2:43 AM, W-Mark Kubacki <span dir="ltr"><<a href="mailto:wmark+nginx@hurrikane.de" target="_blank">wmark+nginx@hurrikane.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">2015-05-05 15:39 GMT+02:00 chen <<a href="mailto:gzchenym@126.com">gzchenym@126.com</a>>:<br>
><br>
> This is v1 of the patchset the implementing the feature SSL Dynamic Record<br>
</span>> Sizing, inspiring by Google Front End […]<br>
><br>
> Any comments is welcome.<br>
<br>
Nice! I've implemented that for Golang in the past and have ported it<br>
to C for you today.<br>
<br>
Although a single initial packet might seem more attractive in<br>
benchmarks, I found that sending two results in better catching parts<br>
of HEAD — which is what we want. Then you will notice some dancing<br>
around IW4, by which we've already sent about 5683 octets. Enough for<br>
me for a making a tradeoff here.<br>
<br>
16k as ssl->buffer_size results in partially filled packets. A better<br>
default value could minimize the overhead (<0.5%) for that trailing<br>
PDUs.<br>
<br>
SSL libraries really should provide a function for computing overhead.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mark<br>
</font></span><br>_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br></blockquote></div><br></div>