Skip to content

schema: validation: correctly validate gzip layer mediatypes#560

Closed
cyphar wants to merge 1 commit intoopencontainers:masterfrom
cyphar:layer-schema-validation
Closed

schema: validation: correctly validate gzip layer mediatypes#560
cyphar wants to merge 1 commit intoopencontainers:masterfrom
cyphar:layer-schema-validation

Conversation

@cyphar
Copy link
Copy Markdown
Member

@cyphar cyphar commented Feb 8, 2017

When the change was made to make gzip optional for layers, the validator
was not updated -- causing completely valid layers to cause warnings
during validation.

Fixes: aad7f24 ("media-types: Define layer media types with and without '+gzip'")
Signed-off-by: Aleksa Sarai asarai@suse.de

@cyphar
Copy link
Copy Markdown
Member Author

cyphar commented Feb 8, 2017

Can we please be a bit more careful?

/cc @wking

When the change was made to make gzip optional for layers, the validator
was not updated -- causing completely valid layers to cause warnings
during validation.

Fixes: aad7f24 ("media-types: Define layer media types with and without '+gzip'")
Signed-off-by: Aleksa Sarai <asarai@suse.de>
@wking
Copy link
Copy Markdown
Contributor

wking commented Feb 8, 2017

Duplicates #548?

@vbatts
Copy link
Copy Markdown
Member

vbatts commented Feb 8, 2017

it is a dup of #548

@vbatts vbatts closed this Feb 8, 2017
@cyphar cyphar deleted the layer-schema-validation branch February 16, 2017 12:02
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.

3 participants