ResponseCspSubscriber probably has more code to deal with parsing libraries for required domains than actually altering the response. This code should be separated out to it's own service.

It may also be beneficial to break out the library parsing into a couple layers
a) A mapping of individual library's policy directive requirements
b) Per-theme aggregate for each policy directive

Comments

gapple created an issue. See original summary.

  • gapple committed c6918e5 on 8.x-1.x
    Issue #2943989: Refactor library parsing to separate service
    
gapple’s picture

Status: Active » Fixed

Caching for each individual library creates a lot of cache items. Will open a followup to use a CacheCollector so that all libraries under each extension only receive one cache item.

Status: Fixed » Closed (fixed)

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