Skip to content

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

Merged
Rom1-B merged 1 commit into
mainfrom
fix_check
Jul 29, 2026
Merged

Fix: enforce report access checks on view and export#373
Rom1-B merged 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
@Rom1-B
Rom1-B merged commit a815f1f into main Jul 29, 2026
3 checks passed
@Rom1-B
Rom1-B deleted the fix_check branch July 29, 2026 06:04
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