Skip to content

fix: check share owner permissions on public share access - #5888

Merged
hacdias merged 1 commit into
filebrowser:masterfrom
kodareef5:fix/share-owner-permission-recheck
Apr 4, 2026
Merged

fix: check share owner permissions on public share access#5888
hacdias merged 1 commit into
filebrowser:masterfrom
kodareef5:fix/share-owner-permission-recheck

Conversation

@kodareef5

Copy link
Copy Markdown
Contributor

Summary

  • Verify that the share owner still has Share and Download permissions when a public share link is accessed
  • Previously, withHashFile checked that the share and user existed but did not re-validate permissions after revocation
  • Adds two test cases for revoked Share and Download permissions

Ref: GHSA-v9w4-gm2x-6rvf

Verify that the share owner still has Share and Download permissions
when a public share link is accessed. Previously, withHashFile only
checked that the share existed and the user existed, but did not
re-validate permissions. This meant shares remained accessible after
an admin revoked the owner's Share or Download permissions.

Ref: GHSA-v9w4-gm2x-6rvf
@kodareef5
kodareef5 requested a review from a team as a code owner April 4, 2026 15:11
@hacdias
hacdias merged commit 7dbf7a3 into filebrowser:master Apr 4, 2026
7 checks passed
kumaraguru1735 pushed a commit to kumaraguru1735/filebrowser-laravel that referenced this pull request Apr 8, 2026
- filebrowser#5891: Check download perm for binary files in resourceGet
- filebrowser#5889: Reject .. paths before realpath (defense in depth)
- filebrowser#5888: Verify share owner download perm on public access
- filebrowser#5848: Normalize double slashes in TUS upload paths
- Add X-Content-Type-Options for epub/svg/html inline previews

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants