Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[20.10 backport] Check the length of the correct variable #42039#42046

Merged
tiborvass merged 1 commit into
moby:20.10moby/moby:20.10from
thaJeztah:20.10_labels_regex_length_checkthaJeztah/docker:20.10_labels_regex_length_checkCopy head branch name to clipboard
Feb 23, 2021
Merged

[20.10 backport] Check the length of the correct variable #42039#42046
tiborvass merged 1 commit into
moby:20.10moby/moby:20.10from
thaJeztah:20.10_labels_regex_length_checkthaJeztah/docker:20.10_labels_regex_length_checkCopy head branch name to clipboard

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport of #42044

- What I did

- How I did it

- How to verify it

- Description for the changelog

Closes #42039

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Nathan Carlson <carl4403@umn.edu>
(cherry picked from commit 8d73c1a)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
Member Author

@cpuguy83 @tiborvass PTAL (this was a bug in a new feature in 20.10)

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah changed the title Check the length of the correct variable #42039 [20.10 backport] Check the length of the correct variable #42039 Feb 18, 2021
@tiborvass
tiborvass merged commit dc1606a into moby:20.10 Feb 23, 2021
@thaJeztah
thaJeztah deleted the 20.10_labels_regex_length_check branch February 23, 2021 18:11
Sign up for free to 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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.