Skip to content

Fix known issues with the CSP

Dominic Couture requested to merge fix-csp-issue into master

What does this MR do?

It fixes known issues with the default CSP (closes #331810 (closed))

  • Allow blob: and data: in worker-src
  • Add the snowplow host to connect-src when it's configured
  • Add a nonce to link rel="preload" tags due to what seems like a Chrome browser bug? In any case it doesn't hurt the other browsers and it fixes the issue

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Dominic Couture

Merge request reports