majestrate / XD

meme bittorrent client
https://xd-torrent.github.io/
MIT License
241 stars 25 forks source link

API call ListTorrentDetailed #30

Closed l-n-s closed 7 years ago

l-n-s commented 7 years ago

Need detailed torrent list, for displaying front page of webui. Right now API is getting spammed with n requests (for every torrent) each second, it may cause problems when n > 100.

majestrate commented 7 years ago

what about a ListManyTorrentInfos function?

On Thu, Sep 21, 2017 at 08:01:56PM +0000, l-n-s wrote:

Need detailed torrent list, for displaying front page of webui. Right now API is getting spammed with n requests (for every torrent) each second, it may cause problems when n > 100.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, [1]view it on GitHub, or [2]mute the thread.

Reverse link: [3]unknown

References

Visible links

  1. https://github.com/majestrate/XD/issues/30
  2. https://github.com/notifications/unsubscribe-auth/AAefxZT2LQzMeh1LvlXJ0HOA6h4uHQU4ks5sksC0gaJpZM4Pf112
  3. https://github.com/majestrate/XD/issues/30
l-n-s commented 7 years ago

IMO better to request all available torrent infos, and filter them by status. You suggest to request many torrent infos by providing their IDs?

majestrate commented 7 years ago

You suggest to request many torrent infos by providing their IDs?

yes

l-n-s commented 7 years ago

We were discussing this lately, RPC command via POST ListTorrentDetailed (analogy of verbose ListTorrent command), returns detailed list of all known torrents.

majestrate commented 7 years ago

fixed in https://github.com/majestrate/XD/commit/6b9e38ffe30367a2f400ef8834bbca39835b6925