From 6e6abe1f4401dffd38c662964e0f94cd5615fadf Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Wed, 13 Jul 2016 20:57:22 -0400 Subject: [PATCH] Temporarily skip resume tests in TLS 1.3. Resumption is not yet implemented. Change-Id: I7c3df2912456a0e0d5339d7b0b1f5819f958e900 Reviewed-on: https://boringssl-review.googlesource.com/8770 Reviewed-by: David Benjamin Commit-Queue: David Benjamin CQ-Verified: CQ bot account: commit-bot@chromium.org --- ssl/test/runner/runner.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/ssl/test/runner/runner.go b/ssl/test/runner/runner.go index 7c86553a..e5494bab 100644 --- a/ssl/test/runner/runner.go +++ b/ssl/test/runner/runner.go @@ -4286,7 +4286,14 @@ func addExtensionTests() { func addResumptionVersionTests() { for _, sessionVers := range tlsVersions { + // TODO(davidben,svaldez): Implement resumption in TLS 1.3. + if sessionVers.version >= VersionTLS13 { + continue + } for _, resumeVers := range tlsVersions { + if resumeVers.version >= VersionTLS13 { + continue + } cipher := TLS_RSA_WITH_AES_128_CBC_SHA if sessionVers.version >= VersionTLS13 || resumeVers.version >= VersionTLS13 { // TLS 1.3 only shares ciphers with TLS 1.2, so