• Johnny's avatar
    fix: add access control checks for attachments, comments, and reactions · c7b48b80
    Johnny authored
    Security fixes for multiple authorization bypass vulnerabilities:
    
    - GetAttachment: Add visibility check via checkAttachmentAccess helper
    - UpdateAttachment: Add ownership check (creator or admin only)
    - Fileserver: Require creator/admin auth for unlinked attachments
    - ListMemoAttachments: Add memo visibility check
    - CreateMemoComment: Add memo visibility check for target memo
    - ListMemoReactions: Add memo visibility check
    - UpsertMemoReaction: Add memo visibility check
    
    All checks follow the existing pattern used in GetMemo for consistency.
    c7b48b80
fileserver.go 21 KB