Skip to content
Closed
Changes from all commits
Commits
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
21 changes: 21 additions & 0 deletions fixes/issue-133-cross-tenant-document-access.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
diff --git a/api/apps/sdk/doc.py b/api/apps/sdk/doc.py
--- a/api/apps/sdk/doc.py
+++ b/api/apps/sdk/doc.py
@@ -18,7 +18,7 @@

from quart import send_file

-from api.apps import login_required
+from api.apps import current_user, login_required
from api.db.db_models import Document, Task
from api.db.joint_services.tenant_model_service import get_model_config_by_id, get_model_config_by_type_and_name, get_tenant_default_model_by_type
from api.db.services.doc_metadata_service import DocMetadataService
@@ -152,6 +152,8 @@
"""
if not document_id:
return get_error_data_result(message="Specify document_id please.")
+ if not DocumentService.accessible(document_id, current_user.id):
+ return get_error_data_result(message="Document not found!")
doc = DocumentService.query(id=document_id)
if not doc:
return get_error_data_result(message=f"The dataset not own the document {document_id}.")