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

Remove yggtorrent support #8871

Closed
wants to merge 4 commits into from
Closed

Remove yggtorrent support #8871

wants to merge 4 commits into from

Conversation

HLFH
Copy link

@HLFH HLFH commented Jun 5, 2020

Hi,

YggTorrent is no longer supported because of Cloudflare protections.
The current issue should be closed and classified as won't fix.

Before Jackett is able to bypass Cloudflare protections thanks to some new contributors, it is best to remove the yggtorrent files as it is no longer supported.

In the mean time, I recommend everyone to use their own scraper for yggtorrent or - more easily - to migrate to an alternative such as the torrent tracker that has recently been implemented into Jackett: ONLYscene.

@ngosang
Copy link
Member

ngosang commented Jun 6, 2020

You need to do some changes in the updater code too.
Anyway this issue is affecting more sites, Jackett developers have to make a global decision about Cloudflare.

@HLFH
Copy link
Author

HLFH commented Jun 7, 2020

Not sure what you mean by the required changes in the updater code.

On a different project, the Cloudflare bypass issue has never been fixed.
wallabag is an alternative to the Pocket read-it-later service: wallabag/wallabag#1399 ; and five years laters, neither the core team nor the contributors fixed it.

Happily, more developers have worked in the last years to tackle this issue, so I guess Jackett will be able to fix it thanks to its community.

@Ekhozie
Copy link

Ekhozie commented Jun 8, 2020

I don't think this PR is still required as cloudflare has been removed from yggtorrent (see #8840) and everything is now working well.

@HLFH
Copy link
Author

HLFH commented Jun 8, 2020

@Ekhozie Yes,

I don't think this PR is still required as cloudflare has been removed from yggtorrent (see #8840) and everything is now working well.

I understand it and it works. I'll close this PR as soon as the discussion is over, that the #8840 issue is closed, and that a global issue concerning the bypass of CloudFlare Security Protections is submitted.

@cadatoiva
Copy link
Collaborator

One thing we could do to remain relatively lightweight and still solve the JS challenges relatively easily is to have a node.js component to the system and run that as the challenge solver: https://stackoverflow.com/questions/15136952/can-a-c-sharp-application-communicate-with-node-js-code

Since it's already a JS engine, it should be able to run the code from the site to parse, and then pass along the finished result.

Other option is to look for progress on the javascript interpreter front.

@ngosang
Copy link
Member

ngosang commented Jun 8, 2020

@Guisch I have been working in a NodeJS server based on this code #8840 (comment) but it's generic, it works with all sites and challenges. And a C# handler/interceptor that is able to send the requests to the NodeJS server and get the cookies. Implements this #8840 (comment)

All is implemented and working but it's no reliable because Cloudflare is able to detect Selenium and Puppeteer and It shows the hCaptcha instead of the JS challenge.

I can share all the code but I think this is going to be the cat and mouse game again...

@Aditya94A
Copy link

I can share all the code but I think this is going to be the cat and mouse game again...

Please do, we gotta start somewhere 😁

@ngosang
Copy link
Member

ngosang commented Jun 9, 2020

This is the server source code => https://github.com/ngosang/FlareSolverr
I will publish the C# interceptor source code soon.

@Guisch @ All Any help to improve the code or to bypass Cloudflare detection will be appreciated.

@ngosang
Copy link
Member

ngosang commented Jun 21, 2020

For those interested in the Cloudflare problem that affects other trackers see #9029

@ngosang ngosang closed this Jun 21, 2020
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.

None yet

5 participants