Skip to content

test: fix renegotiation tests in pummel

Rodrigo Muino Tomonari requested to merge github/fork/Trott/fix-ish into master

With a recent OpenSSL update to core, a change in the behavior of s_client result in pummel test failures because they spam with renegotiation requests before the renegotiation is complete. Modify tests to reduce initial spamming.

This fixes the TLS renegotiation test. While it fixes the bug in the HTTPS test too, that is still failing with the OpenSSL update for other (too-be-determined) reasons.

Refs: https://github.com/nodejs/node/pull/25381#issuecomment-456822479

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports