Skip to content

Misc updates for making CSP headers work#98

Merged
bheesham merged 7 commits intomozilla-iam:masterfrom
bheesham:misc-updates-for-csp
Oct 17, 2025
Merged

Misc updates for making CSP headers work#98
bheesham merged 7 commits intomozilla-iam:masterfrom
bheesham:misc-updates-for-csp

Conversation

@bheesham
Copy link
Contributor

Turns out the ingress controller was not acting on the nginx.ingress.kubernetes.io/configuration-snippet annotation because we didn't configure it to (or perhaps the defaults changed).

This also sets up some rules to deal with mozilla-iam/dino-park-front-end#716.

Jira: IAM-1775

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates CSP (Content Security Policy) headers configuration to work with the ingress controller by enabling snippet annotations and restructuring security header implementation.

  • Enables snippet annotations in the ingress controller configuration
  • Restructures CSP headers from a single line to multiple separate header directives
  • Removes public/private environment conditional logic and related routing rules

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
k8s/values/test.yaml Removes public flag from test environment configuration
k8s/values.yaml Removes public flag from default values
k8s/templates/router.yaml Removes conditional public routing logic
k8s/templates/public-splitter.yaml Updates location patterns and removes CSP header, simplifies logout redirect logic
k8s/templates/ingress.yaml Restructures CSP and security headers from single line to multiple separate directives
k8s/templates/ingress-controller.yaml Enables snippet annotations and adds events RBAC permissions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bheesham bheesham merged commit 0d5fdc7 into mozilla-iam:master Oct 17, 2025
1 check passed
@bheesham bheesham deleted the misc-updates-for-csp branch October 17, 2025 19:30
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