Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: allow-list auth0-client header for forwarding#2668

Merged
tusharpandey13 merged 3 commits into
mainfrom
feat/proxy-header-allowlist-auth0-client
May 21, 2026
Merged

feat: allow-list auth0-client header for forwarding#2668
tusharpandey13 merged 3 commits into
mainfrom
feat/proxy-header-allowlist-auth0-client

Conversation

@tusharpandey13
Copy link
Copy Markdown
Contributor

@tusharpandey13 tusharpandey13 commented May 18, 2026 •
edited
Loading

Add auth0-client to DEFAULT_HEADER_ALLOW_LIST in proxy utility.
This enables sending adoption telemetry from universal components.

Changes

  • utils/proxy.ts : add "auth0-client" to the forwarding allow-list
  • src/utils/proxy.test.ts : 5 new tests covering forwarding, casing, absence, coexistence with other headers, and value preservation

@tusharpandey13
feat(proxy): allow-list auth0-client header for forwarding
bb38bf1 
UI Components telemetry header was stripped in RWA proxy mode,
causing zero visibility for all proxy-routed traffic. Unblocks
universal-components team from shipping adoption analytics.
@tusharpandey13 tusharpandey13 requested a review from a team as a code owner May 18, 2026 17:12
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

This PR adds auth0-client telemetry header to the proxy forwarding allow-list and provides comprehensive test coverage validating case-insensitive forwarding, no unwanted injection when absent, forwarding alongside other headers, and preservation of explicitly supplied values.

Changes

Auth0 Client Telemetry Header Forwarding

Layer / File(s) Summary
Allow-list auth0-client header
src/utils/proxy.ts		 DEFAULT_HEADER_ALLOW_LIST now includes "auth0-client" to enable forwarding of the telemetry header through proxied requests.
Test auth0-client header forwarding
src/utils/proxy.test.ts		 Comprehensive test cases validate that auth0-client is forwarded when present (case-insensitive), not injected when absent, forwarded alongside other allow-listed headers, and preserves explicitly supplied values without SDK injection.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A telemetry header hops on through,
Case-insensitive, tried and true,
Never injected where it shouldn't be,
Tests ensure it flows so free!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding the auth0-client header to an allow-list for forwarding in proxy mode.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/proxy-header-allowlist-auth0-client

Comment @coderabbitai help to get the list of available commands and usage tips.

@tusharpandey13 tusharpandey13 changed the title feat(proxy): allow-list auth0-client header for forwarding feat: allow-list auth0-client header for forwarding May 18, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 18, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.85%. Comparing base (fba47c9) to head (0d1b99f).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #2668   +/-   ##
=======================================
  Coverage   89.85%   89.85%           
=======================================
  Files          72       72           
  Lines        8809     8810    +1     
  Branches     1874     1875    +1     
=======================================
+ Hits         7915     7916    +1     
  Misses        881      881           
  Partials       13       13

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

rax7389
rax7389 approved these changes May 19, 2026
View reviewed changes
Copy link
Copy Markdown

@rax7389 rax7389 left a comment

Choose a reason for hiding this comment

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

LGTM

@tusharpandey13 tusharpandey13 merged commit 6454e60 into main May 21, 2026
10 checks passed
@tusharpandey13 tusharpandey13 deleted the feat/proxy-header-allowlist-auth0-client branch May 21, 2026 06:01
@Piyush-85 Piyush-85 mentioned this pull request May 22, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Piyush-85 Piyush-85 Piyush-85 approved these changes

+1 more reviewer

@rax7389 rax7389 rax7389 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tusharpandey13 @codecov-commenter @rax7389 @Piyush-85