Skip to content

fix: WPB-25750 Enable passing cspExtraConnectSrc value to nginx-ingress-services#5236

Open
mohitrajain wants to merge 4 commits into
developfrom
wpb-25750-cspExtraConnect-multiIng
Open

fix: WPB-25750 Enable passing cspExtraConnectSrc value to nginx-ingress-services#5236
mohitrajain wants to merge 4 commits into
developfrom
wpb-25750-cspExtraConnect-multiIng

Conversation

@mohitrajain
Copy link
Copy Markdown
Contributor

@mohitrajain mohitrajain commented May 21, 2026

Enable passing cspExtraConnectSrc value to nginx-ingress-services when working with multi-ingresses (renderCSPInIngress=True). It would be required for webapp to connect to third party sft servers.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

…ss-services when working with multi-ingresses
@mohitrajain mohitrajain requested review from a team as code owners May 21, 2026 06:54
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label May 21, 2026
@mohitrajain mohitrajain changed the title fix: WPB-25750 Enable passing cspExtraConnectSrc value to nginx-ingre… fix: WPB-25750 Enable passing cspExtraConnectSrc value to nginx-ingress-services May 21, 2026
@smatting
Copy link
Copy Markdown
Contributor

Approved. Note that I introduced a failure here which will make the integration tests fail for this PR.

I think we should ignore this failure and merge this PR anyway, because

  • this change in this PR doesn't need to be reflected in wire-ingress just yet, because wire-ingress doesn't support multi-ingress yet. there is a separate ticket for that.
  • the integration tests don't cover the change of this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants