Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
test-invalid-server-name-extension-resumption.py: test improvements
session resume with malformed SNI:
  Corrected to expect an illegal parameter alert for
  a malformed SNI.

session resume with different SNI:
  Corrected to not expect session resumption when a
  different SNI is given.
  • Loading branch information
Nikos Mavrogiannopoulos committed Nov 27, 2017
1 parent a9b27d1 commit 412caa9
Showing 1 changed file with 3 additions and 18 deletions.
21 changes: 3 additions & 18 deletions scripts/test-invalid-server-name-extension-resumption.py
Expand Up @@ -189,23 +189,8 @@ def main():
node = node.add_child(ClientHelloGenerator(
ciphers,
extensions=ext))
node = node.add_child(ExpectServerHello(
extensions={ExtensionType.renegotiation_info:None},
resume=True))
node = node.add_child(ExpectCertificate())
node = node.add_child(ExpectServerHelloDone())
node = node.add_child(ClientKeyExchangeGenerator())
node = node.add_child(ChangeCipherSpecGenerator())
node = node.add_child(FinishedGenerator())
node = node.add_child(ExpectChangeCipherSpec())
node = node.add_child(ExpectFinished())
node = node.add_child(ApplicationDataGenerator(
bytearray(b"GET / HTTP/1.0\n\n")))
node = node.add_child(ExpectApplicationData())
node = node.add_child(AlertGenerator(AlertLevel.warning,
AlertDescription.close_notify))
node = node.add_child(ExpectAlert(AlertLevel.warning,
AlertDescription.close_notify))
node = node.add_child(ExpectAlert(AlertLevel.fatal,
AlertDescription.illegal_parameter))
node.next_sibling = ExpectClose()
node = node.add_child(ExpectClose())

Expand Down Expand Up @@ -252,7 +237,7 @@ def main():
extensions=ext))
node = node.add_child(ExpectServerHello(
extensions={ExtensionType.renegotiation_info:None},
resume=True))
resume=False))
node = node.add_child(ExpectCertificate())
node = node.add_child(ExpectServerHelloDone())
node = node.add_child(ClientKeyExchangeGenerator())
Expand Down

0 comments on commit 412caa9

Please sign in to comment.