Skip to content

Escape CSV output#6089

Merged
schlessera merged 2 commits intomainfrom
add/csv-escaping
May 6, 2025
Merged

Escape CSV output#6089
schlessera merged 2 commits intomainfrom
add/csv-escaping

Conversation

@schlessera
Copy link
Member

This PR adds default escaping of CSV output to avoid inadvertent execution of CSV formulas.

@codecov
Copy link

codecov bot commented May 6, 2025

Codecov Report

Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
php/utils.php 91.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@schlessera schlessera merged commit bdc2e5d into main May 6, 2025
52 checks passed
@schlessera schlessera deleted the add/csv-escaping branch May 6, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant