Skip to content

Commit a51cadc

Browse files
Add defensive file-object guard in session upload flows
Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>
1 parent 812d28d commit a51cadc

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

hyperbrowser/client/managers/async_manager/session.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,10 @@ async def upload_file(
183183
original_error=exc,
184184
) from exc
185185
else:
186+
if file_obj is None:
187+
raise HyperbrowserError(
188+
"file_input must be a file path or file-like object"
189+
)
186190
files = {"file": file_obj}
187191
response = await self._client.transport.post(
188192
self._client._build_url(f"/session/{id}/uploads"),

hyperbrowser/client/managers/sync_manager/session.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,10 @@ def upload_file(
175175
original_error=exc,
176176
) from exc
177177
else:
178+
if file_obj is None:
179+
raise HyperbrowserError(
180+
"file_input must be a file path or file-like object"
181+
)
178182
files = {"file": file_obj}
179183
response = self._client.transport.post(
180184
self._client._build_url(f"/session/{id}/uploads"),

0 commit comments

Comments
 (0)