Skip to content

Fix: enforce report access checks on view and export - #373

Open
Rom1-B wants to merge 1 commit into
mainfrom
fix_check
Open

Fix: enforce report access checks on view and export#373
Rom1-B wants to merge 1 commit into
mainfrom
fix_check

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

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

  • It fixes #N/A
  • Viewing or exporting a report now correctly checks the per-profile access rights and the report's enabled state instead of ignoring them.
  • The report rights page now requires the right level that matches the write actions it performs.
  • Dashboard and preference actions now check that the entry belongs to the current user before changing it, and a list size taken from the request is now safely converted to a number.
  • A rendered value is now properly escaped, a redirect file name is sanitized, and CSV exports no longer corrupt cell content while guarding against a leading formula character.

Screenshots (if appropriate):

@Rom1-B
Rom1-B requested a review from MyvTsv July 28, 2026 13:19
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