Skip to content

Fix EnvoyFilter port validation#3688

Merged
istio-testing merged 2 commits intomasterfrom
fix-envoyfilter-port-limit
Apr 11, 2026
Merged

Fix EnvoyFilter port validation#3688
istio-testing merged 2 commits intomasterfrom
fix-envoyfilter-port-limit

Conversation

@keithmattix
Copy link
Copy Markdown
Contributor

@keithmattix keithmattix commented Apr 10, 2026

Apparently the missing quotation causes trunctation. Fixes istio/istio#59832

Signed-off-by: Keith Mattix II <keithmattix@microsoft.com>
@keithmattix keithmattix requested a review from a team as a code owner April 10, 2026 19:58
@istio-policy-bot
Copy link
Copy Markdown

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?

Courtesy of your friendly test nag.

@istio-policy-bot
Copy link
Copy Markdown

😊 Welcome @keithmattix! This is either your first contribution to the Istio api repo, or it's been
a while since you've been here.

You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 10, 2026
@keithmattix keithmattix added cherrypick/release-1.29 Set this label on a PR to auto-merge it to the release-1.29 branch and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 10, 2026
Signed-off-by: Keith Mattix II <keithmattix@microsoft.com>
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 10, 2026
Copy link
Copy Markdown
Member

@howardjohn howardjohn left a comment

Choose a reason for hiding this comment

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

That is... fun behavior

@istio-testing istio-testing merged commit 92fcfbe into master Apr 11, 2026
5 checks passed
@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: new pull request created: #3689

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherrypick/release-1.29 Set this label on a PR to auto-merge it to the release-1.29 branch size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

truncated upper port value '6553' in validation for EnvoyFilter v1alpha3

5 participants