Skip to content

Allow empty string in server-tokens annotation for NGINX Plus #2125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

lucacome
Copy link

  • An empty value with NGINX OSS will now return annotations.nginx.org/server-tokens: Invalid value: "": must be a boolean instead of default/cafe-ingress was rejected: with error: annotations.nginx.org/server-tokens: Required value
  • An empty value with NGINX Plus is valid.

@lucacome lucacome requested a review from a team October 26, 2021 01:22
@lucacome lucacome self-assigned this Oct 26, 2021
@lucacome lucacome requested review from pleshakov and soneillf5 and removed request for a team October 26, 2021 01:22
@github-actions github-actions bot added the bug An issue reporting a potential bug label Oct 26, 2021
@lucacome lucacome requested a review from ciarams87 October 26, 2021 15:52
@lucacome lucacome changed the title Allow empty string in server-tokens annotation Allow empty string in server-tokens annotation for NGINX Plus Oct 26, 2021
@lucacome lucacome merged commit 943404d into master Oct 26, 2021
@lucacome lucacome deleted the fix/server-tokens-validation branch October 26, 2021 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue reporting a potential bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants