Skip to content

demos/tests/run_tests.py: enable sertificate verification #3833

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Dec 6, 2023
Merged

demos/tests/run_tests.py: enable sertificate verification #3833

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2616c63
demos/tests/run_tests.py: enable sertificate verification
Wovchena Aug 14, 2023
d7c69c9
Merge branch 'master' into demos/tests/run_tests.py-enable-sertificat…
Wovchena Sep 1, 2023
622d469
demos/tests/run_tests.py: remove import ssl
Wovchena Sep 1, 2023
93c8898
Merge branch 'master' into demos/tests/run_tests.py-enable-sertificat…
Wovchena Oct 10, 2023
a7c81b9
Update run_tests.py
mpushki Oct 11, 2023
af979e0
Merge branch 'master' into demos/tests/run_tests.py-enable-sertificat…
Wovchena Dec 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 4 additions & 6 deletions demos/tests/run_tests.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,12 @@
"""

import argparse
import certifi
import contextlib
import csv
import json
import os
import shlex
import ssl
import subprocess # nosec B404 # disable import-subprocess check
import sys
import tempfile
Expand Down Expand Up @@ -241,11 +241,9 @@ def main():

print(f"{len(demos_to_test)} demos will be tested:")
print(*[demo.subdirectory for demo in demos_to_test], sep =',')

no_verify_because_of_windows = ssl.create_default_context()
no_verify_because_of_windows.check_hostname = False
no_verify_because_of_windows.verify_mode = ssl.CERT_NONE
with urlopen(COCO128_URL, context=no_verify_because_of_windows) as zipresp: # nosec B310 # disable urllib_urlopen because url is hardcoded
os.environ["REQUESTS_CA_BUNDLE"] = certifi.where()
os.environ["SSL_CERT_FILE"] = certifi.where()
with urlopen(COCO128_URL) as zipresp: # nosec B310 # disable urllib_urlopen because url is hardcoded
with ZipFile(BytesIO(zipresp.read())) as zfile:
zfile.extractall(args.test_data_dir)

Expand Down