Skip to content

Clarify that 416 is valid on a blob put#593

Open
sudo-bmitch wants to merge 1 commit intoopencontainers:mainfrom
sudo-bmitch:pr-blob-put-invalid-range
Open

Clarify that 416 is valid on a blob put#593
sudo-bmitch wants to merge 1 commit intoopencontainers:mainfrom
sudo-bmitch:pr-blob-put-invalid-range

Conversation

@sudo-bmitch
Copy link
Contributor

Fixes #590.

Signed-off-by: Brandon Mitchell <git@bmitch.net>
@jcarter3
Copy link

jcarter3 commented Jan 9, 2026

I'll publicly log my opinion that while I agree this is a valid response code, I disagree that it's a MUST, especially given backwards compatibility with existing registries in production today. I'd rather push to say that in chunked upload, the final put should not have data at all.

@sudo-bmitch
Copy link
Contributor Author

I'll publicly log my opinion that while I agree this is a valid response code, I disagree that it's a MUST, and would rather push to say that in chunked upload, the final put should not have data at all.

I think removing support for data in the final chunk is a bigger than this PR. There is language supporting it on the previous lines of the spec that have been there for a while. Can we make that a separate issue/PR?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP status code for a blob PUT with an invalid range

2 participants