d3-dsv icon indicating copy to clipboard operation
d3-dsv copied to clipboard

Make parse compatible with CSP

Open vrana opened this issue 4 years ago • 4 comments

Using parseRows with CSP enabled is quite cumbersome. I don't see a reason why this shouldn't just work (even though a little slower).

vrana avatar Oct 14 '21 08:10 vrana

Is there any reason not to merge it? This allows d3 to work on applications that use CSP without unsafe-eval, which seems quite beneficial. The less-performant branch is only taken if the new Function has failed due to CSP.

koto avatar Mar 02 '22 16:03 koto

I've added a test and fixed the formatting.

vrana avatar Mar 14 '22 14:03 vrana

Just curious, how much slower is this?

curran avatar Nov 05 '22 08:11 curran

Hey, any update on this? It would be really helpful for us as it would allow us to apply proper CSP while still using this library.

zzacharo avatar Aug 31 '23 07:08 zzacharo