Skip to content

rbac: allowlist FilterStateInput for xDS matcher in Network RBAC #39467

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
May 14, 2025

Conversation

agrawroh
Copy link
Member

@agrawroh agrawroh commented May 14, 2025

Description

This PR adds the support for FilterStateInput in the Network RBAC filter.


Commit Message: allowlist FilterStateInput for xDS matcher in Network RBAC filter.
Additional Description: added support for FilterStateInput to be allowed in the xDS matcher inside the Network RBAC filter.
Risk Level: Low
Testing: Added Integration & Unit Tests.
Docs Changes: N/A
Release Notes: Yes
Platform Specific Features: No

Signed-off-by: Rohit Agrawal <rohit.agrawal@databricks.com>
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #39467 was opened by agrawroh.

see: more, trace.

@agrawroh agrawroh marked this pull request as ready for review May 14, 2025 16:33
@agrawroh agrawroh requested a review from yanavlasov as a code owner May 14, 2025 16:33
@agrawroh agrawroh merged commit 8aea316 into envoyproxy:main May 14, 2025
24 checks passed
fishpan1209 pushed a commit to fishpan1209/envoy that referenced this pull request May 22, 2025
…oyproxy#39467)

## Description

This PR adds the support for `FilterStateInput` in the Network RBAC
filter.

---

Commit Message: allowlist `FilterStateInput` for xDS matcher in Network
RBAC filter.
Additional Description: added support for `FilterStateInput` to be
allowed in the xDS matcher inside the Network RBAC filter.
Risk Level: Low
Testing: Added Integration & Unit Tests.
Docs Changes: N/A
Release Notes: Yes
Platform Specific Features: No

Signed-off-by: Rohit Agrawal <rohit.agrawal@databricks.com>
Signed-off-by: Ting Pan <panting@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants