Other people's download failures and progress show on everyone's page #294

Closed
opened 2026-10-19 10:40:16 -07:00 by rays · 1 comment
Owner

web/src/events.ts: download_error toasts 'Download failed: ...' for any feed's failed download, and progress writes 'downloading ' into #count for any feed's download. Every page receives every event, and deadair_done already checks mine(ev.feed), but these two do not. So one person's scheduled download failures, and the names of the files being downloaded, appear on another person's page. Found in a repo audit. Fix: return early when !mine(ev.feed).

web/src/events.ts: download_error toasts 'Download failed: ...' for any feed's failed download, and progress writes 'downloading <file>' into #count for any feed's download. Every page receives every event, and deadair_done already checks mine(ev.feed), but these two do not. So one person's scheduled download failures, and the names of the files being downloaded, appear on another person's page. Found in a repo audit. Fix: return early when !mine(ev.feed).
rays added the bug label 2026-10-19 10:40:16 -07:00
Author
Owner

Fixed in 995ee05: progress and download_error events for a feed not on this person's list are dropped by the page, as deadair_done already was. Deployed in ipx-web.

Fixed in 995ee05: progress and download_error events for a feed not on this person's list are dropped by the page, as deadair_done already was. Deployed in ipx-web.
rays closed this issue 2026-10-19 11:17:39 -07:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: rays/ipx#294