Skip to content

fix: require secret for forwarded user auth#8880

Merged
Startrekzky merged 1 commit into
mainfrom
copilot/fix-vulnerability
May 20, 2026
Merged

fix: require secret for forwarded user auth#8880
Startrekzky merged 1 commit into
mainfrom
copilot/fix-vulnerability

Conversation

@klesh
Copy link
Copy Markdown
Contributor

@klesh klesh commented May 16, 2026

Summary

Reject spoofed X-Forwarded-User headers unless a configured shared secret matches the forwarded secret header.

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. component/framework This issue or PR relates to the framework pr-type/bug-fix This PR fixes a bug labels May 16, 2026
Reject spoofed X-Forwarded-User headers unless a configured shared secret matches the forwarded secret header.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@klesh klesh force-pushed the copilot/fix-vulnerability branch from e9e8529 to 1cd3fb5 Compare May 20, 2026 02:26
Copy link
Copy Markdown
Contributor

@Startrekzky Startrekzky left a comment

Choose a reason for hiding this comment

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

LGTM

@Startrekzky Startrekzky merged commit 7e9a2ae into main May 20, 2026
10 checks passed
@Startrekzky Startrekzky deleted the copilot/fix-vulnerability branch May 20, 2026 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/framework This issue or PR relates to the framework pr-type/bug-fix This PR fixes a bug size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants