Project

General

Profile

Actions

Bug #18890

closed

[sdk] bring the python keep SDK in line with ARVADOS_API_HOST_INSECURE

Added by Ward Vandewege about 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Story points:
-
Release relationship:
Auto

Description

ARVADOS_API_HOST_INSECURE is interpreted in our codebase as "do not validate TLS certificate validity". This includes "do not check if the hostname matches the name on the certificate".

The Python SDK keep code is a bit of an outlier, because it only disables pycurl.SSL_VERIFYPEER when ARVADOS_API_HOST_INSECURE is set, which means the certificate validity check is disabled, but the hostname on the certificate still needs to match the hostname connected to.

libcurl has another flag to disable the hostname check, pycurl.SSL_VERIFYHOST (it's called CURLOPT_SSL_VERIFYHOST in https://curl.se/libcurl/c/curl_easy_setopt.html).

We should also disable pycurl.SSL_VERIFYHOST when ARVADOS_API_HOST_INSECURE is set, for consistency's sake.


Subtasks 1 (0 open1 closed)

Task #18891: review 18890-python-sdk-verifyhostResolvedWard Vandewege03/17/2022Actions
Actions

Also available in: Atom PDF