User pfaffman is experiencing content security policy (CSP) violations when defining DISCOURSE_CDN_URL in their multisite configuration. The browser rejects CDN URLs, resulting in a failed asset load. xrav3nz suggests that CDN URLs should be computed and included in the CSP by default.
After inspecting the CSP header and asset sources, xrav3nz discovers that the issue stems from prioritizing DISCOURSE_S3_CDN_URL over DISCOURSE_CDN_URL for asset generation, even when the latter is explicitly defined. xrav3nz suspects that an additional GlobalSetting.use_s3? check is needed to resolve the issue.
pfaffman attempts to set use_s3 and precompile assets but sees no change, suggesting that the problem may be related to confusion between S3 and local asset storage or their CDN mirrors.
in the env: stanza in my yml for a fairly standard multisite configuration. All of the CDN urls get rejected by the browser as a CSP error.
content security policy script src claims “Additional whitelisted script sources. The current host and CDN are included by default. See Mitigate XSS Attacks with Content Security Policy.”, but when I define it (or add/remove it to discourse.conf and sv restart unicorn), I get this:
even with content security policy report only set to true, the site still won’t load.
Turning off content_security_policy or adding the CDN URL to content security policy script src seems to be required to get the browser to load the assets.
CDN URLs should be computed and included in the CSP by default. Could you also provide (or try comparing) the actual CSP served in the header and the source of the blocked assets?
And here is the URL for one of the assets that won’t load: https://lcsupport-92e2.kxcdn.com/brotli_asset/preload-store-d32dcf974dddcac742f8a7a6aa7fcd686185920b201029d0ecb2b85527ef9034.js
https://abedmulti-92e2.kxcdn.com/uploads/assets/
https://abedmulti-92e2.kxcdn.com/uploads/brotli_asset/
# i.e. DISCOURSE_S3_CDN_URL + /brotli_asset/
But the actual address is
https://lcsupport-92e2.kxcdn.com/brotli_asset/preload-store-d32dcf974dddcac742f8a7a6aa7fcd686185920b201029d0ecb2b85527ef9034.js
# i.e. DISCOURSE_CDN_URL + /brotli_asset/...
The relevant CSP code:
We prioritize using DISCOURSE_S3_CDN_URL for assets when available. This aligns with the CDN asset URL generation.
@pfaffman Does GlobalSetting.use_s3? return true for your site?
I wonder if we need an additional GlobalSetting.use_s3? check here. Does having GlobalSetting.s3_cdn_url necessarily imply GlobalSetting.use_s3?? I’m a bit hazy with asset genereation / S3 CDN now could someone more familiar with it also take a look? Thanks!