Skip to content

Upgrade to reggie v0.6.1 to fix issue with Accept header#400

Merged
jdolitsky merged 1 commit intoopencontainers:mainfrom
jdolitsky:accept-header
Apr 6, 2023
Merged

Upgrade to reggie v0.6.1 to fix issue with Accept header#400
jdolitsky merged 1 commit intoopencontainers:mainfrom
jdolitsky:accept-header

Conversation

@jdolitsky
Copy link
Copy Markdown
Contributor

Resolves #396

Signed-off-by: Josh Dolitsky <josh@dolit.ski>
@sudo-bmitch
Copy link
Copy Markdown
Contributor

@Dylmay PTAL

@Dylmay
Copy link
Copy Markdown

Dylmay commented Apr 5, 2023

Can confirm that it's fixed the problem. The accept header is no longer incorrectly set and the PUT is successful

Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@jdolitsky jdolitsky merged commit e79ab90 into opencontainers:main Apr 6, 2023
@jdolitsky jdolitsky deleted the accept-header branch April 6, 2023 14:20
@jdolitsky jdolitsky mentioned this pull request Apr 19, 2023
@sudo-bmitch sudo-bmitch mentioned this pull request Feb 1, 2024
8 tasks
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.

Conformance Test: 'Accept' header set during manifest put

4 participants