runner: Allow tokbind without RI/EMS in TLS 1.3.

There was a QUIC-specific ALPS bug, so I'm thinking we should loop
addExtensionTests at all protocols. To do so, we need to fix this bug in
the test expectation.

Change-Id: Ic05a4cb2ea32e7145441a0273cd65966c41534ec
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/44986
Reviewed-by: Adam Langley <agl@google.com>
chromium-5359
David Benjamin 4 years ago committed by Adam Langley
parent 41676bfd86
commit f4a88296fc
  1. 2
      ssl/test/runner/handshake_server.go

@ -1475,7 +1475,7 @@ func (hs *serverHandshakeState) processClientExtensions(serverExtensions *server
serverExtensions.tokenBindingVersion = config.TokenBindingVersion
}
if len(hs.clientHello.tokenBindingParams) > 0 && (!hs.clientHello.extendedMasterSecret || hs.clientHello.secureRenegotiation == nil) {
if c.vers < VersionTLS13 && len(hs.clientHello.tokenBindingParams) > 0 && (!hs.clientHello.extendedMasterSecret || hs.clientHello.secureRenegotiation == nil) {
return errors.New("client sent Token Binding without EMS and/or RI")
}

Loading…
Cancel
Save