silk-framework / silk

Silk Linked Data Integration Framework
http://silkframework.org/
Other
240 stars 62 forks source link

Bugfix: CSV injection #811

Closed edufuga closed 1 month ago

edufuga commented 2 months ago

Bugfix: CSV Injection

This PR adds a sanitizer for CSV rows. It deals with the causes for CSV Injection, as documented in https://owasp.org/www-community/attacks/CSV_Injection.

The CsvParser from Univocity can't be extended (it's final) in a "sanitizing subclass of CsvParser", so the solution simply maps over the cells and converts them explicitly.