Skip to content

Commit 8fb03e8

Browse files
committed
fix: add missing verify=VERIFY_SSL to network requests
fetch_app_key and fetch_wrapped_key_share were missing the verify=VERIFY_SSL parameter on their requests.get() calls, making them inconsistent with all other HTTP calls in the file and causing SSL failures with self-signed certificates.
1 parent 0be8027 commit 8fb03e8

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/phase/utils/network.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ def fetch_app_key(token_type: str, app_token, host) -> str:
124124

125125
URL = f"{host}/service/secrets/tokens/"
126126

127-
response = requests.get(URL, headers=headers)
127+
response = requests.get(URL, headers=headers, verify=VERIFY_SSL)
128128

129129
if response.status_code != 200:
130130
raise ValueError(f"Request failed with status code {response.status_code}: {response.text}")
@@ -164,7 +164,7 @@ def fetch_wrapped_key_share(token_type: str, app_token: str, host: str) -> str:
164164

165165
URL = f"{host}/service/secrets/tokens/"
166166

167-
response = requests.get(URL, headers=headers)
167+
response = requests.get(URL, headers=headers, verify=VERIFY_SSL)
168168

169169
if response.status_code != 200:
170170
raise ValueError(f"Request failed with status code {response.status_code}: {response.text}")

0 commit comments

Comments
 (0)