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

Defender clocked the exe as being malacious. #25

Open
Sharkaboi opened this issue Apr 29, 2020 · 5 comments
Open

Defender clocked the exe as being malacious. #25

Sharkaboi opened this issue Apr 29, 2020 · 5 comments
Assignees
Labels
announcement Announcement about an issue

Comments

@Sharkaboi
Copy link

Was downloading Jojo from gogoanime since about 10 hrs,
Settings if it helps : 720p, ep 1-23 , stardust egypt arc.
Running 4 Threads, encountered two non HLS links and started two ffmpeg instances and I think it crashed/was terminated after one ffmpeg download finished (only one file was completed).

image

@Oshan96
Copy link
Owner

Oshan96 commented Apr 29, 2020

Can you tell me the version you're using?

And about the antivirus software flagging this as a trojan is a false positive. I have addressed this separately in multiple issues but I should address this in README as well.

This happens because of the subprocess statements (to run FFMPEG) in the code (you can see them in source code) mostly. Apart from that, the other reason is because of the the use of Cypto libraries to decrypt encrypted URLs from websites like Twist.

The subprocess call will be removed in a later version release with the removal of FFMPEG dependency (currently working on it and is optional in monkey-dl version, but didn't remove it completely soon) and hopefully the false positive torjan alert will be solved

@Oshan96 Oshan96 pinned this issue Apr 29, 2020
@Oshan96 Oshan96 self-assigned this Apr 29, 2020
@Sharkaboi
Copy link
Author

Running latest v1.04 .
Ye figured it was a ffmpeg issue, I knew it wasn't actually a Trojan but saw the whole process was killed and the exe was missing and found it to be quarantined .

Also just a suggestion, but there's an api for whole collection from horriblesubs if you want that as an option.

@Oshan96
Copy link
Owner

Oshan96 commented Apr 29, 2020

Can you provide me the link to the API? So that I'll look into it. As long as it provides direct links, I will be able to plug it in monkey-dl.

PS: Once I came along an API for HorribleSubs, but the provided download links (direct) were hard to retrieve since it had all sorts of limited downloads per hour and such. Not sure you're referring to the same :D

@Sharkaboi
Copy link
Author

This one used web torrent i think, needs node js.
https://pypi.org/project/horrible-downloader/
Their direct download servers are premium i think, and they all have bandwidth limits.
They also have this option in their website, but i guess it's not that easily of a system to integrate.
https://horriblesubs.info/irc-guide/

@Oshan96
Copy link
Owner

Oshan96 commented Apr 30, 2020

Yup. This is the API I came across, and I've checked the wrapper also. The thing is monkey-dl does not have a torrent client integrated within; and I do not want to have external dependencies on this as as well (that's why I took care of FFMPEG dependency as well). I have checked for pure python based torrent client API's and found some promising ones as well. If either one of them works as to my expectations, I might integrate a torrent client as well so I will be using the horriblesub magnet links from their API to download the files.

As for now I'm only looking for the direct links and yeah, as stated, horriblesubs direct downloads are in premium servers and it's an issue as of now. If someone finds a way for this and does a PR with the fixes for that I will be glad to merge it :)

Repository owner locked as resolved and limited conversation to collaborators May 6, 2020
@Oshan96 Oshan96 added the announcement Announcement about an issue label May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
announcement Announcement about an issue
Projects
None yet
Development

No branches or pull requests

2 participants