Skip to content

Files API: [PUT] Check the content of the json payload#11855

Closed
tino097 wants to merge 25 commits into
mindsdb:releases/25.11.1from
tino097:bugfix/file-api-traversal
Closed

Files API: [PUT] Check the content of the json payload#11855
tino097 wants to merge 25 commits into
mindsdb:releases/25.11.1from
tino097:bugfix/file-api-traversal

Conversation

@tino097

@tino097 tino097 commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Description

Small refactor of the code with adding helpers
Added safe_extract for zipfiles
Updated the test scenarios

(Please delete options that are not relevant)

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 📄 This change requires a documentation update

Verification Process

To ensure the changes are working as expected:

  • Test Location: tests/unit/api/http/files_test.py

Checklist:

  • My code follows the style guidelines(PEP 8) of MindsDB.
  • I have appropriately commented on my code, especially in complex areas.
  • Relevant unit and integration tests are updated or added.

@tino097 tino097 changed the title Check the content of the json payload Files API: [PUT] Check the content of the json payload Nov 5, 2025
@tino097
tino097 marked this pull request as ready for review November 11, 2025 18:44
@breinero
breinero requested review from StpMax and ea-rus November 21, 2025 16:56
Comment thread mindsdb/api/http/namespaces/file.py
Comment thread mindsdb/api/http/namespaces/file.py
@tino097
tino097 requested a review from a team as a code owner November 28, 2025 15:53
@tino097
tino097 changed the base branch from develop to main November 28, 2025 15:54
@tino097
tino097 changed the base branch from main to releases/25.11.1 November 28, 2025 15:54
@tino097
tino097 changed the base branch from releases/25.11.1 to main December 2, 2025 17:55
@tino097
tino097 changed the base branch from main to releases/25.11.1 December 2, 2025 17:55
@ea-rus
ea-rus deleted the branch mindsdb:releases/25.11.1 December 3, 2025 08:51
@ea-rus ea-rus closed this Dec 3, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 3, 2025
@martyna-mindsdb

Copy link
Copy Markdown
Contributor

Hey @tino097
Thank you for contributing to MindsDB! Unfortunately, due to the fact that your PR was targeting the releases/25.11.1 branch, it was automatically closed when merging this release to main. Therefore, we need to request you to please recreate this PR and point it to the main branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants