Skip to content

Commit cf4cd4f

Browse files
Merge pull request #50 from kurtjallo/fix/async-delete-validation
fix: add missing ID validation in AsyncDocuments.delete
2 parents 71e7008 + 0c74c0b commit cf4cd4f

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

moorcheh_sdk/resources/documents.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -777,6 +777,10 @@ async def delete(
777777
f"Attempting to delete {len(ids)} document(s) from namespace"
778778
f" '{namespace_name}' with IDs: {ids}"
779779
)
780+
if not all(isinstance(item_id, (str, int)) and item_id for item_id in ids):
781+
raise InvalidInputError(
782+
"All items in 'ids' list must be non-empty strings or integers."
783+
)
780784

781785
response_data = await self._client._request(
782786
method="POST",

0 commit comments

Comments
 (0)