search

opencontainers / distribution-spec

OCI Distribution Specification
https://opencontainers.org
Apache License 2.0
781 stars 201 forks source link

clarify where OCI-Chunk-Min-Length response headers are expected #481

Closed mvdan closed 7 months ago

mvdan commented 9 months ago

(see commit message)

rogpeppe commented 9 months ago

From a practical point of view, I suspect the GET response to find the current offset should also return the chunk length, to better support stateless client resumption of uploads, but I've no idea if that's actually a current convention or not.

sudo-bmitch commented 9 months ago

The idea of setting this on the POST was that it's the first request, before any PATCH requests that would exceed the size. A client doing a chunked upload should use this for all subsequent requests of that blob.

mvdan commented 8 months ago

Should I do anything else to get this tiny clarification merged? Hopefully eight reviews aren't actually necessary :)

sudo-bmitch commented 8 months ago

Should I do anything else to get this tiny clarification merged? Hopefully eight reviews aren't actually necessary :)

It just needs a second maintainer to approve. Copying the commit message into the PR may get more attention.