Skip to content

[GHSA-ghc4-35x6-crw5] Envoy has RBAC Header Validation Bypass via Multi-Value Header Concatenation#7331

Open
sekveaja wants to merge 1 commit intosekveaja/advisory-improvement-7331from
sekveaja-GHSA-ghc4-35x6-crw5
Open

[GHSA-ghc4-35x6-crw5] Envoy has RBAC Header Validation Bypass via Multi-Value Header Concatenation#7331
sekveaja wants to merge 1 commit intosekveaja/advisory-improvement-7331from
sekveaja-GHSA-ghc4-35x6-crw5

Conversation

@sekveaja
Copy link
Copy Markdown

@sekveaja sekveaja commented Apr 8, 2026

Hello,
I notice that Patched version is not aligned between GHSA-ghc4-35x6-crw5 and
GHSA-ghc4-35x6-crw5

@github
Copy link
Copy Markdown
Collaborator

github commented Apr 8, 2026

Hi there @phlax! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to sekveaja/advisory-improvement-7331 April 8, 2026 15:24
@phlax
Copy link
Copy Markdown

phlax commented Apr 8, 2026

@sekveaja i think this does need a fix - but im not sure this pr fixes

not familiar with this json spec - but i think you need to bump all the last_affected fields patch+1

@sekveaja
Copy link
Copy Markdown
Author

sekveaja commented Apr 8, 2026

@phlax it is the first time I report the discrepancy as an user. I'm not sure where to bump as suggested.

@phlax
Copy link
Copy Markdown

phlax commented Apr 8, 2026

its new to me also - but i think the change needs to happen to last_affected - as suggested above - same file - different fields - updating the timestamp wont do anything afaict

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.

3 participants