Other people's download failures and progress show on everyone's page #294
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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).
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.