-
Notifications
You must be signed in to change notification settings - Fork 647
feat: Add SchemeHeaderTransformation to ClientTrafficPolicy #7980
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: jukie <[email protected]>
✅ Deploy Preview for cerulean-figolla-1f9435 canceled.
|
Signed-off-by: jukie <[email protected]>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7980 +/- ##
=======================================
Coverage 72.82% 72.82%
=======================================
Files 237 237
Lines 35475 35495 +20
=======================================
+ Hits 25835 25851 +16
- Misses 7801 7805 +4
Partials 1839 1839 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: jukie <[email protected]>
Signed-off-by: jukie <[email protected]>
|
/retest |
|
Nice Yeah it's the same and I can close mine in favor of it but that one is missing the explicit override field. |
|
Correction, this is a listener level setting so adding to BTP isn't right. |
yeah can we highlight this in the other issue, and steer it in the right direction |
What this PR does / why we need it:
Adds the ability to configure SchemeHeaderTransformation inside ClientTrafficPolicy
Which issue(s) this PR fixes:
Fixes #7979
Release Notes: Yes
I validate that this change makes it possible to run a gRPC .NET service with gateway TLS termination