feat(upload): Revert use of multipart uploads - #6324
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5daa4f1. Configure here.
| response = upload_something(relay, project_id, project_key) | ||
|
|
||
| assert response.status_code == 504 | ||
| assert response.status_code == 500 # not 504 |
There was a problem hiding this comment.
Timeout test no longer checks timeouts
Medium Severity
test_objectstore_timeout now sets timeout instead of stream_timeout, so the TUS stream never hits the one-second deadline. The mock then raises NotImplementedError, and the test asserts 500 rather than the previous 504 gateway timeout.
Reviewed by Cursor Bugbot for commit 5daa4f1. Configure here.
1 participant
Multipart is getting replaced by resumable uploads on the Objectstore side. Remove its traces.
Fixes INGEST-1048