Skip to content

Fix(Core): harden report data handling and prevent cross-entity expos… - #386

Merged
RomainLvr merged 1 commit into
10.0/bugfixesfrom
some_fix_v10
Sep 4, 2026
Merged

Fix(Core): harden report data handling and prevent cross-entity expos…#386
RomainLvr merged 1 commit into
10.0/bugfixesfrom
some_fix_v10

Conversation

@stonebuzz

Copy link
Copy Markdown
Contributor

…ure (#381)

Backport of PR #381 from main: escape chart labels/links via json_encode() to prevent stored XSS, cast state filter values to int before SQL interpolation, validate f_name against an allow-list before building the PNG export path, and remove the logs activity report (instance-wide counts with no entity restriction).

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

backport for #381

  • Escape chart labels before injecting them into report graphs
  • Cast state filter values before building SQL
  • Remove Logs distribution report to prevent cross-entity data display

Screenshots (if appropriate):

…ure (#381)

Backport of PR #381 from main: escape chart labels/links via
json_encode() to prevent stored XSS, cast state filter values to int
before SQL interpolation, validate f_name against an allow-list before
building the PNG export path, and remove the logs activity report
(instance-wide counts with no entity restriction).

Co-authored-by: Stanislas Kita <7335054+stonebuzz@users.noreply.github.com>
@stonebuzz
stonebuzz requested a review from RomainLvr September 4, 2026 07:28
@RomainLvr
RomainLvr merged commit 8d0ed45 into 10.0/bugfixes Sep 4, 2026
3 checks passed
@RomainLvr
RomainLvr deleted the some_fix_v10 branch September 4, 2026 08:01
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.

2 participants