Skip to content

compose file: disallow additional properties in credential_spec - #1657

Merged
silvin-lubecki merged 1 commit into
docker:masterfrom
thaJeztah:compose_credential_spec_fix
Feb 6, 2019
Merged

compose file: disallow additional properties in credential_spec#1657
silvin-lubecki merged 1 commit into
docker:masterfrom
thaJeztah:compose_credential_spec_fix

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

image: busybox
credential_spec:
File: "/foo"
file: "/foo"

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

looks like this test was incorrect? Or is there a way to make these case-insensitive?

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztahforce-pushed the compose_credential_spec_fix branch from eb3998b to 1e99ed3CompareFebruary 2, 2019 17:28
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #1657 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@ Coverage Diff @@## master #1657 +/- ##
==========================================
+ Coverage 56.11% 56.13% +0.01% 
==========================================
Files 306 306 Lines 20909 20909 ==========================================
+ Hits 11734 11737 +3 + Misses 8328 8325 -3 
Partials 847 847

@vdemeestervdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🐯

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki
silvin-lubecki merged commit b877ef8 into docker:masterFeb 6, 2019
@GordonTheTurtleGordonTheTurtle added this to the 19.03.0 milestone Feb 6, 2019
@thaJeztah
thaJeztah deleted the compose_credential_spec_fix branch February 6, 2019 15:44
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@thaJeztah@codecov-io@vdemeester@silvin-lubecki@GordonTheTurtle