Skip to content

Expand allowed status codes#171

Closed
pmengelbert wants to merge 1 commit intoopencontainers:mainfrom
bloodorangeio:expand-allowed-status-codes
Closed

Expand allowed status codes#171
pmengelbert wants to merge 1 commit intoopencontainers:mainfrom
bloodorangeio:expand-allowed-status-codes

Conversation

@pmengelbert
Copy link
Copy Markdown
Contributor

@pmengelbert pmengelbert commented Jul 10, 2020 •
edited
Loading

POST /v2/<name>/blobs/uploads/?digest=<digest> should allow for 202 Accepted.
All DELETE calls should allow 405 Method Not Allowed.

This PR resolves the following comment by @jdolitsky on issue 68. Resolving this issue will make several registries go green on the relevant workflows.

@pmengelbert
Expand allowed status codes
33a44f4 
POST /v2/<name>/blobs/uploads/?digest=<digest> should allow for 202 Accepted.
All DELETE calls should allow 405 Method Not Allowed.

Signed-off-by: Peter Engelbert <pmengelbert@gmail.com>
@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Jul 10, 2020 •
edited by caniszczyk
Loading

LGTM

Approved with PullApprove

@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Jul 10, 2020

Should be noted that the current spec is not as permissive, but it will be factored into the rewrite

@jdolitsky jdolitsky mentioned this pull request Jul 10, 2020
Closed
3 tasks
This was referenced Jul 22, 2020
Closed
Closed
@wy65701436 wy65701436 mentioned this pull request Jul 31, 2020
Closed
@vbatts
Copy link
Copy Markdown
Member

vbatts commented Aug 18, 2020

glad we're testing for this.
I recall the discussion for 202 being used in addition to the 201 written in the spec.
Is there other background on using 202 Accepted that should be linked here? or update to the spec needed to relate to this conformance test?

@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Aug 24, 2020

@vbatts its something that should be accounted for in the spec reorg: #178

@pmengelbert
Copy link
Copy Markdown
Contributor Author

pmengelbert commented Aug 24, 2020

@vbatts This has been included in the latest commit for PR #178

@vbatts
Copy link
Copy Markdown
Member

vbatts commented Sep 16, 2020

the maintainers had chatted about this in a call recently. While everyone noted that some of these response codes are being used just to mimic docker registry, they also noted that they really ought to be corrected in the original registry implementation.

@vbatts
Copy link
Copy Markdown
Member

vbatts commented Sep 16, 2020

I'm not opposed to this conformance test, but might we track the conversation to have this aligned in the community, rather than only widening the conformance to make it green for more registries?

@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Oct 5, 2020

FWIW this change is now reflected in the spec.md on master branch

@jdolitsky jdolitsky mentioned this pull request Oct 14, 2020
Open
@pmengelbert pmengelbert mentioned this pull request Oct 29, 2020
Merged
@vbatts
Copy link
Copy Markdown
Member

vbatts commented Jan 28, 2021

@jdolitsky are saying that this PR is no longer needed? or just that the spec has the language supporting this?

@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Jan 28, 2021

@vbatts - the spec already contains language, so this PR should prob be merged

Base automatically changed from master to main March 9, 2021 17:38
@jdolitsky
Copy link
Copy Markdown
Contributor

jdolitsky commented Jun 23, 2021

closing this. we settled on final HTTP codes for 1.0

@jdolitsky jdolitsky closed this Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jdolitsky jdolitsky jdolitsky approved these changes

+1 more reviewer

@renmaosheng renmaosheng renmaosheng approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pmengelbert @jdolitsky @vbatts @renmaosheng