Skip to content

API for api key auth forwarding setting #6444

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

zhaohuabing
Copy link
Member

@zhaohuabing zhaohuabing commented Jul 1, 2025

This is required when

  • the client identity needs to be used in a downstream filter or the backend service.
  • we want to avoid exposing the API key to the backend service.

API for: #6461

@zhaohuabing zhaohuabing requested a review from a team as a code owner July 1, 2025 13:58
@zhaohuabing zhaohuabing force-pushed the api-key-auth-forward-client branch from 5611930 to d64b17f Compare July 1, 2025 14:04
Copy link

codecov bot commented Jul 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.11%. Comparing base (fc2766b) to head (90aeea0).
Report is 15 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6444      +/-   ##
==========================================
+ Coverage   70.78%   71.11%   +0.32%     
==========================================
  Files         220      220              
  Lines       37820    37892      +72     
==========================================
+ Hits        26772    26947     +175     
+ Misses       9488     9377     -111     
- Partials     1560     1568       +8     

☔ 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.

@zhaohuabing
Copy link
Member Author

/retest

@arkodg arkodg added this to the v1.5.0-rc.1 Release milestone Jul 9, 2025
@zhaohuabing zhaohuabing self-assigned this Jul 15, 2025
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
@zhaohuabing zhaohuabing force-pushed the api-key-auth-forward-client branch from 2a0b56f to fa604ac Compare July 16, 2025 03:00
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
@zhaohuabing zhaohuabing force-pushed the api-key-auth-forward-client branch from fa604ac to d1bee6f Compare July 16, 2025 03:14
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
@zhaohuabing zhaohuabing requested a review from arkodg July 17, 2025 03:58
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
Signed-off-by: Huabing (Robin) Zhao <[email protected]>
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