Skip to content

Commit

Permalink
Remove twitter.com from endpoint handshake test for OpenSSL 1.0.2 (#3038
Browse files Browse the repository at this point in the history
)

* Remove failing endpoint

* Remove twitter.com only for OpenSsl 1.0.2

* Removing twitter for all
  • Loading branch information
WesleyRosenblum authored Sep 10, 2021
1 parent 8c52f6a commit 01b3c46
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions tests/integration/s2n_client_endpoint_handshake_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,13 @@
# permissions and limitations under the License.
#

import argparse
import os
import subprocess
import sys
import time
import socket
import subprocess
import itertools
import argparse
from s2n_test_constants import *

from s2n_test_constants import *

# If a cipher_preference_version is specified, we will use it while attempting the handshake;
# otherwise, s2n will use the default. If an expected_cipher is specified, the test will pass
Expand All @@ -32,7 +30,12 @@
{"endpoint": "facebook.com"},
{"endpoint": "google.com"},
{"endpoint": "s3.amazonaws.com"},
{"endpoint": "twitter.com"},
# twitter.com offers RSA certificates even when the client does not include RSA PSS
# in the the Signature Schemes extension. Disabling twitter for now since this prevents
# s2n from negotiating a handshake if the libcrypto does not support RSA PSS signature
# algorithms with RSA Certificates
# See https://github.com/aws/s2n-tls/pull/3030
# {"endpoint": "twitter.com"},
{"endpoint": "wikipedia.org"},
{"endpoint": "yahoo.com"},
]
Expand Down

0 comments on commit 01b3c46

Please sign in to comment.