Skip to content

Correctly escape report name in CSV exports #20785

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 25, 2023
Merged

Correctly escape report name in CSV exports #20785

merged 1 commit into from
May 25, 2023

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented May 25, 2023

Description:

Backport of #20782 to Matomo 4

Review

@sgiehl sgiehl added the c: Security For issues that make Matomo more secure. Please report issues through HackerOne and not in Github. label May 25, 2023
@sgiehl sgiehl added this to the 4.14.x milestone May 25, 2023
@sgiehl sgiehl merged commit 32814d3 into 4.x-dev May 25, 2023
@sgiehl sgiehl deleted the csvescape4x branch May 25, 2023 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
c: Security For issues that make Matomo more secure. Please report issues through HackerOne and not in Github.
Milestone
4.15.0
Development

Successfully merging this pull request may close these issues.
1 participant
@sgiehl