Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated torrents-csv URL #276

Merged

Conversation

ducalex
Copy link
Contributor

@ducalex ducalex commented Apr 23, 2024

The previous url torrents-csv.ml seems to redirect to malware now.

I'm not 100% sure torrents-csv.com is the official replacement but it seems legit and returns correct results.

@ducalex ducalex force-pushed the ducalex/update-torrents-csv-url branch from 9269a41 to f8e6d31 Compare April 23, 2024 14:21
@xavier2k6
Copy link
Member

@ducalex Any screenshot of torrents-csv.com working??, I tried your changes & it's not working via qBittorrent in-built search but if I search via chrome browser....it works?

The previous url `torrents-csv.ml` seems to redirect to malware now.
@ducalex ducalex force-pushed the ducalex/update-torrents-csv-url branch from f8e6d31 to 784f600 Compare June 23, 2024 18:30
@ducalex
Copy link
Contributor Author

ducalex commented Jun 23, 2024

That's embarrassing, I don't remember if I tested in qBittorrent! I fixed the code and I now confirm that it works. The torrent description link doesn't work but I don't think torrents-csv.com offers one.

image

@ducalex
Copy link
Contributor Author

ducalex commented Jun 23, 2024

Also the current plugin only fetches the first page of results. torrents-csv.com seems to limit that to 100 results which might not be enough? I'd be happy to add multi-page support to the plugin but that's probably out of scope for this PR.

@xavier2k6
Copy link
Member

The torrent description link doesn't work

There's another plugin that has the same problem iirc & eztv doesn't work either, there is a PR open for that but haven't tested the proposed changes there.

@xavier2k6 xavier2k6 merged commit bd2119b into qbittorrent:master Jun 27, 2024
6 checks passed
@xavier2k6
Copy link
Member

@ducalex Thank You!

@ducalex ducalex deleted the ducalex/update-torrents-csv-url branch July 2, 2024 17:01
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