yonlu/yellowjacket
1
1
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects Releases 13 Wiki Activity

slskd: a peer that has queued us is abandoned after 10 minutes, whatever its queue position #275

New Issue
Open
opened 2026-09-27 01:58:36 +00:00 by yonlu · 1 comment
yonlu commented 2026-09-27 01:58:36 +00:00
Owner
Copy Link
Copy Source

Since #263 a grab is abandoned when no bytes move for 10 minutes. On Soulseek the usual reason no bytes move is that the peer has queued us — Queued, Remotely — and a peer at position 2 is worth waiting for while one at position 400 is not. The timer cannot tell them apart.

slskd exposes the position: Transfer.placeInQueue, refreshed by GET /api/v0/transfers/downloads/{username}/{id}/position, which asks the peer.

Direction

While a wanted file is remotely queued, ask its position periodically. Abandon at once when the position is beyond what could clear in reasonable time; treat a position that has improved as progress, so the stall clock restarts; keep an overall ceiling so a queue that moves one place an hour does not hold the grab all day. The fallback from #263 then moves to the next peer.

Since #263 a grab is abandoned when no bytes move for 10 minutes. On Soulseek the usual reason no bytes move is that the peer has queued us — `Queued, Remotely` — and a peer at position 2 is worth waiting for while one at position 400 is not. The timer cannot tell them apart. slskd exposes the position: `Transfer.placeInQueue`, refreshed by `GET /api/v0/transfers/downloads/{username}/{id}/position`, which asks the peer. ## Direction While a wanted file is remotely queued, ask its position periodically. Abandon at once when the position is beyond what could clear in reasonable time; treat a position that has improved as progress, so the stall clock restarts; keep an overall ceiling so a queue that moves one place an hour does not hold the grab all day. The fallback from #263 then moves to the next peer.
yonlu added the Area/DownloadsPlatform/Desktop
Priority
Medium
3
Kind/Enhancement labels 2026-09-27 01:58:36 +00:00
yonlu self-assigned this 2026-09-27 01:58:54 +00:00
yonlu added the
Status
In Progress
 label 2026-09-27 01:58:54 +00:00
yonlu commented 2026-09-27 01:58:55 +00:00
Author
Owner
Copy Link
Copy Source

Batch branch batch/274-276-slskd with #274-#276, one PR. See the issue body for the approach.

Batch branch batch/274-276-slskd with #274-#276, one PR. See the issue body for the approach.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Area/Design
Area/Downloads
Area/Explore
Area/Library-UI
Area/Metadata
Area/Packaging
Area/Player
Area/Queue
Area/Settings
Area/Shell-Nav
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
 Platform/Android
Platform/Desktop
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
In Progress
Somebody is actively working on this right now
No labels Area/Downloads Kind/Enhancement Platform/Desktop
Priority
Medium
3
Status
In Progress
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
logan yonlu
No Assignees yonlu
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: yonlu/yellowjacket#275