Skip to content
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

[GEOS-11346] Enable strict Content-Security-Policy by default #8167

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sikeoka
Copy link
Contributor

@sikeoka sikeoka commented Dec 26, 2024

Using the Content-Security-Policy-Report-Only header was necessary to support the Wicket 9 upgrade but the Content-Security-Policy header which blocks CSP violations can be used now that most of the remaining CSP violations, particularly the modal window, have been resolved. This will provide a more secure default mode, make any remaining CSP violations (if there are any left) more obvious and ensure that developers do not add new CSP violations.

Checklist

For core and extension modules:

  • New unit tests have been added covering the changes.
  • Documentation has been updated (if change is visible to end users).
  • The REST API docs have been updated (when changing configuration objects or the REST controllers).
  • There is an issue in the GeoServer Jira (except for changes that do not affect administrators or end users in any way).
  • Commit message(s) must be in the form [GEOS-XYZWV] Title of the Jira ticket.
  • Bug fixes and small new features are presented as a single commit.
  • Each commit has a single objective (if there are multiple commits, each has a separate JIRA ticket describing its goal).

@sikeoka
Copy link
Contributor Author

sikeoka commented Jan 10, 2025

@jodygarnett do you have thoughts about this?

@jodygarnett
Copy link
Member

jodygarnett commented Jan 10, 2025

I would like to see this enabled on main branch, and we can tell folks about the setting in the release notes.

For this specific PR please update the docs upgrade page to describe the change and double check the setting is mentioned on the application property page.

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