Problem/Motivation

The style-src key in the CSP settings is missing in seckit.settings.yml.

Proposed resolution

The schema is already defined, just need to add the key to the config install.

CommentFileSizeAuthor
#2 3366124-style-src-key-missing.patch360 bytesjnlar

Comments

jnlar created an issue. See original summary.

jnlar’s picture

jnlar
Primary language English
Location Sydney, Australia
commented
StatusFileSize
new 3366124-style-src-key-missing.patch360 bytes

Attached patch to add style-src to the config install file

mfb’s picture

mfb
they or he
commented
Status: Active » Reviewed & tested by the community

LGTM. For completeness the default value should be saved if it's missing, but I guess anyone using this module would configure it?

jnlar’s picture

jnlar
Primary language English
Location Sydney, Australia
commented

Thanks @mfb. Yeah, anyone using the module should have it set to '' if they've saved config (with/without setting style-src). AFAIK the only edge case is a fresh install and then interacting with the config via code.

  • mcdruid committed 29159686 on 2.x authored by jnlar 
    Issue #3366124 by jnlar, mfb: style-src key missing in seckit.settings....
mcdruid’s picture

mcdruid commented
Version: 2.0.1 » 2.x-dev
Status: Reviewed & tested by the community » Fixed

Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.