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

Fix(zoro): update the extractor in fetchEpisodeSources and update zoro domain. #482

Merged
merged 16 commits into from
Mar 4, 2024

Conversation

carlosesteven
Copy link
Contributor

@carlosesteven carlosesteven commented Mar 2, 2024

URL: /anime/zoro/watch?episodeId=...

Change class: rapidcloud.ts to: megacloud.ts

Reason: rapidcloud.ts class used static keys from a repository that no longer work. I replaced it with a class that extracts the decryption keys from the same code.

Use as a base this extractor code and adapt it to the consumet.ts

Did you add tests for your changes?

No, tests already exist.

Summary

Other information

dist/extractors/megacloud.js Dismissed Show dismissed Hide dismissed
src/extractors/megacloud.ts Dismissed Show dismissed Hide dismissed
dist/extractors/megacloud.js Dismissed Show dismissed Hide dismissed
src/extractors/megacloud.ts Dismissed Show dismissed Hide dismissed
server: number;
};

class MegaCloud {
Copy link
Contributor

@riimuru riimuru Mar 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you extend VideoExtractor here so then you can override the extract function and use this.client.get instead of axios.get, use other extractors as a reference.

Copy link
Contributor Author

@carlosesteven carlosesteven Mar 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the new commit.

default?: boolean;
};

type intro_outro = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This type isn't needed, use Intro imported from models instead.

Copy link
Contributor Author

@carlosesteven carlosesteven Mar 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the new commit.

megacloud.script.concat(Date.now().toString())
);

text = data;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change to const test = data;

Copy link
Contributor Author

@carlosesteven carlosesteven Mar 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the new commit.

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.

3 participants