Skip to content

Conversation

@pajay-rao
Copy link
Contributor

Description

Fixes nil pointer error that happens in envoy.go due to traffic redirection

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@pajay-rao pajay-rao requested review from a team as code owners November 5, 2025 06:38
@pajay-rao pajay-rao added backport/1.22 Changes are backported to 1.22 backport/ent/1.22 Changes are backported to 1.22 ent labels Nov 5, 2025
@github-actions github-actions bot added the theme/cli Flags and documentation for the CLI interface label Nov 5, 2025
Copy link
Contributor

@anilvpatel anilvpatel left a comment

Choose a reason for hiding this comment

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

LGTM

@pajay-rao pajay-rao added the pr/no-changelog PR does not need a corresponding .changelog entry label Nov 5, 2025
Copy link
Contributor

@anandmukul93 anandmukul93 left a comment

Choose a reason for hiding this comment

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

lgtm

@pajay-rao pajay-rao merged commit 42f828a into main Nov 5, 2025
215 of 227 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/ent/1.22 Changes are backported to 1.22 ent backport/1.22 Changes are backported to 1.22 pr/no-changelog PR does not need a corresponding .changelog entry theme/cli Flags and documentation for the CLI interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants