search

cloudflare / tls-tris

crypto/tls, now with 100% more 1.3. THE API IS NOT STABLE AND DOCUMENTATION IS NOT GUARANTEED.
Other
292 stars 50 forks source link

(test) Moves NSS interoperability tests to allowed failures #66

Closed kriskwiatkowski closed 6 years ago

kriskwiatkowski commented 6 years ago

Tests against NSS master branch do not work anymore correctly. Reported error says: 1: SSL3[1768679856]: handle alert record 1: SSL3[1768679856] received alert, level = 2, description = 70 SSL: destroy sid: sid=0x696c27a0 cached=0 1: SSL[1768679856]: handshake gathering, rv=-1 1: SSL[1768679856]: SecureSend: returning -1 count, error -12190 tstclnt: write to SSL socket failed: SSL_ERROR_PROTOCOL_VERSION_ALERT: Peer reports incompatible or unsupported protocol version.

Not sure what's the root cause of the problem, but as the last commit on tls-tris has succeeded, the problem/change causing regression must be on NSS side. I propose to temporarily allow failures of those tests to keep tls-tris build regression less.

Lekensteyn commented 6 years ago

Thanks, but this was already addressed in #61 which is waiting to get merged.

Edit: now it is merged