Skip to content

Conversation

@maxbrunet
Copy link
Contributor

@maxbrunet maxbrunet commented Oct 30, 2025

What type of PR is this?

Feature

What this PR does / why we need it:

Add support for ExtAuth context extensions in SecurityPolicy:

spec:
  extAuth:
    contextExtensions:
    - name: hello
      inline: world
    - name: foo
      type: ValueRef
      valueRef:
        kind: ConfigMap
        name: my-cm
        key: bar
    - name: bar
      type: ValueRef
      valueRef:
        kind: Secret
        name: my-secret
        key: foo

Which issue(s) this PR fixes:

Closes #6592

Release Notes: Yes

@maxbrunet maxbrunet requested a review from a team as a code owner October 30, 2025 20:37
@maxbrunet maxbrunet force-pushed the feat/extauth/context-extensions branch 5 times, most recently from e1d8c4c to d485d82 Compare October 30, 2025 21:18
@codecov
Copy link

codecov bot commented Oct 30, 2025

Codecov Report

❌ Patch coverage is 81.87500% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.36%. Comparing base (e5c1bb3) to head (0939490).
⚠️ Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
internal/provider/kubernetes/controller.go 60.97% 12 Missing and 4 partials ⚠️
internal/gatewayapi/securitypolicy.go 93.10% 2 Missing and 2 partials ⚠️
internal/provider/kubernetes/indexers.go 92.30% 2 Missing ⚠️
internal/xds/translator/credentialInjector.go 66.66% 0 Missing and 1 partial ⚠️
internal/xds/translator/custom_response.go 66.66% 0 Missing and 1 partial ⚠️
internal/xds/translator/extauth.go 93.33% 0 Missing and 1 partial ⚠️
internal/xds/translator/extproc.go 66.66% 0 Missing and 1 partial ⚠️
internal/xds/translator/lua.go 66.66% 0 Missing and 1 partial ⚠️
internal/xds/translator/oidc.go 66.66% 0 Missing and 1 partial ⚠️
internal/xds/translator/wasm.go 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7383      +/-   ##
==========================================
+ Coverage   72.30%   72.36%   +0.06%     
==========================================
  Files         231      231              
  Lines       33939    34071     +132     
==========================================
+ Hits        24539    24655     +116     
- Misses       7632     7640       +8     
- Partials     1768     1776       +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.

@maxbrunet maxbrunet force-pushed the feat/extauth/context-extensions branch from d485d82 to 0939490 Compare October 30, 2025 23:22
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.

ExtAuth: allow passing context extensions

1 participant