d3 / d3-dsv

A parser and formatter for delimiter-separated values, such as CSV and TSV.
https://d3js.org/d3-dsv
ISC License
437 stars 76 forks source link

Make parse compatible with CSP #87

Open vrana opened 3 years ago

vrana commented 3 years ago

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).

koto commented 2 years ago

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.

vrana commented 2 years ago

I've added a test and fixed the formatting.

curran commented 2 years ago

Just curious, how much slower is this?

zzacharo commented 1 year ago

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.