Skip to content

Fix/csv formula sanitization#27

Open
lucas-amiaud wants to merge 2 commits intomasterfrom
fix/csv-formula-sanitization
Open

Fix/csv formula sanitization#27
lucas-amiaud wants to merge 2 commits intomasterfrom
fix/csv-formula-sanitization

Conversation

@lucas-amiaud
Copy link
Member

To prevent CSV formula injection, Windmill automatically sanitizes values starting with =, +, -, @, \t or \r by prefixing them with a single quote '.

This behavior is enabled by default, but can be disabled with fluid API

@lucas-amiaud lucas-amiaud requested a review from amanteaux March 6, 2026 12:23
@github-actions
Copy link

github-actions bot commented Mar 6, 2026

File Coverage [56.92%]
CsvExporter.java 96.33% 🍏
ExportCsvConfig.java 36.07%
Total Project Coverage 70.28%

@coveralls
Copy link

Coverage Status

coverage: 75.03% (+0.3%) from 74.71%
when pulling 0efdbe2 on fix/csv-formula-sanitization
into cb6af3e on master.

/** If true, values starting with =, +, -, @, \t, \r will be prefixed with ' to prevent CSV formula injection */
@Builder.Default private final boolean sanitizeFormulas = true;
/** The field names for which formula sanitization should be disabled */
@Builder.Default private final Set<String> fieldNamesExcludedFromSanitization = Collections.emptySet();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est un peu overkill non?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants