Skip to content

fix(security): secure redirect handler - remove auth headers on insecure redirect#222

Closed
MIchaelMainer wants to merge 1 commit intomainfrom
mmainer/redirect-sec
Closed

fix(security): secure redirect handler - remove auth headers on insecure redirect#222
MIchaelMainer wants to merge 1 commit intomainfrom
mmainer/redirect-sec

Conversation

@MIchaelMainer
Copy link
Copy Markdown
Member

Improve redirect logic to remove authorization headers on insecure redirect

  • Cookie and Authorization header removal on insecure redirect
  • Removes Proxy-Authorization when:
    • No proxy is configured
    • Proxy is bypassed for the redirect destination
    • This prevents unnecessary credential removal
  • Customizable Scrubbing - Users can implement custom logic via ScrubSensitiveHeaders callback for application-specific headers

@MIchaelMainer MIchaelMainer requested a review from a team as a code owner February 28, 2026 02:26
@sonarqubecloud
Copy link
Copy Markdown

@baywet
Copy link
Copy Markdown
Member

baywet commented Mar 2, 2026

Closing as duplicate of #221

@baywet baywet closed this Mar 2, 2026
@github-project-automation github-project-automation bot moved this to Done ✔️ in Kiota Mar 2, 2026
@baywet baywet deleted the mmainer/redirect-sec branch March 2, 2026 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants