Skip to content

feat(upload): Disallow creation with upload#5734

Merged
jjbayer merged 7 commits intomasterfrom
feat/upload-pure-create
Mar 19, 2026
Merged

feat(upload): Disallow creation with upload#5734
jjbayer merged 7 commits intomasterfrom
feat/upload-pure-create

Conversation

@jjbayer
Copy link
Member

@jjbayer jjbayer commented Mar 17, 2026

No longer allow TUS "creation with upload", so we have full control over the ingestion path of the actual file upload.

Fixes: INGEST-803

@linear-code
Copy link

linear-code bot commented Mar 17, 2026

@jjbayer jjbayer changed the title Feat/upload pure create feat(upload): Disallow creation with upload Mar 17, 2026
Comment on lines +190 to +198
10,
400,
"stream shorter than lower bound: received 9 < 10",
"stream shorter than lower bound: received 10 < 11",
id="smaller_than_announced",
),
pytest.param(
11,
12,
400,
"stream exceeded upper bound: received 11 > 10",
"stream exceeded upper bound: received 12 > 11",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm using a predefined upload location below, which happens to have a length of 11.

@jjbayer jjbayer marked this pull request as ready for review March 18, 2026 09:38
@jjbayer jjbayer requested a review from a team as a code owner March 18, 2026 09:38
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@jjbayer jjbayer added this pull request to the merge queue Mar 19, 2026
Merged via the queue into master with commit 576de04 Mar 19, 2026
30 checks passed
@jjbayer jjbayer deleted the feat/upload-pure-create branch March 19, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants