Skip to content

fix(rtorrent): fall back to XML-RPC for torrent list fetch#1098

Merged
trim21 merged 3 commits intojesec:masterfrom
awsms:fix/rtorrent-empty-list
Apr 18, 2026
Merged

fix(rtorrent): fall back to XML-RPC for torrent list fetch#1098
trim21 merged 3 commits intojesec:masterfrom
awsms:fix/rtorrent-empty-list

Conversation

@awsms
Copy link
Copy Markdown
Contributor

@awsms awsms commented Apr 17, 2026

Description

Some rTorrent instances return JSON-RPC parse errors (-32700) for the d.multicall2 torrent list request, even though other RPC calls still succeed. This caused Flood to silently fall back to an empty torrent list while transfer stats and add-torrent operations kept working.

Retry the torrent list fetch over XML-RPC when that specific JSON-RPC failure occurs, keeping the existing behavior for all other calls.

Types of changes

  • Breaking change (changes that break backward compatibility of public API or CLI - semver MAJOR)
  • New feature (non-breaking change which adds functionality - semver MINOR)
  • Bug fix (non-breaking change which fixes an issue - semver PATCH)

Some rTorrent instances return JSON-RPC parse errors (-32700) for
the d.multicall2 torrent list request, even though other RPC calls
still succeed. This caused Flood to silently fall back to an empty
torrent list while transfer stats and add-torrent operations kept
working.

Retry the torrent list fetch over XML-RPC when that specific JSON-RPC
failure occurs, keeping the existing behavior for all other calls.
@trim21
Copy link
Copy Markdown
Collaborator

trim21 commented Apr 17, 2026

I don't quiet understand, you are linking this PR in #631, are we talking about Deluge or rtorrent?

@trim21
Copy link
Copy Markdown
Collaborator

trim21 commented Apr 17, 2026

OK, I know.

I need you to add this pr description as comment to the code so we don't need to use git blame to understand why we are doing this.

@awsms
Copy link
Copy Markdown
Contributor Author

awsms commented Apr 17, 2026

I need you to add this pr description as comment to the code so we don't need to use git blame to understand why we are doing this.

Done, I've also referenced the rTorrent issue #.

@trim21 trim21 enabled auto-merge (squash) April 18, 2026 04:48
@trim21 trim21 merged commit ff6870d into jesec:master Apr 18, 2026
23 checks passed
trim21 pushed a commit to trim21/flood that referenced this pull request May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants