pystardust / ani-cli

A cli tool to browse and play anime
GNU General Public License v3.0
7.63k stars 533 forks source link

Major outage - script doesn't work #1161

Closed Raif1 closed 1 year ago

Raif1 commented 1 year ago

Metadata (please complete the following information) Version: [e.g. 2.0.2] OS: Gentoo Linux Shell: Zsh Anime: Any

Describe the bug printf: 0x#1: invalid hex number .local/bin/ani-cli: line 147: warning: command substitution: ignored null byte in input Episode not released.` Steps To Reproduce

  1. Run `ani-cli '
  2. Search any anime.
  3. Choose any episode

Expected behavior The errors should not be present. The episodes are supposed to load.

Screenshots (if applicable; you can just drag the image onto github) image image

Additional context I installed mpv, aria2, FFmpeg, and fzf with Emerge, and they were installed system-wide. However, I only installed ani-cli for my user. Could this be the cause of the error I am currently experiencing? Everything worked fine yesterday. But I'm facing the errors now.

net528890 commented 1 year ago

i got the same issue i have been tried re install from yay , git clone from git , and use on releases, it all doesn't work

uzuto commented 1 year ago

same error here

justchokingaround commented 1 year ago

da677bd9f242de5e11b08f8ad421c26566a81336 could be used as a short term fix, if we don't figure out how the encryption works quickly enough

Abhijith021 commented 1 year ago

same issue is there anything we can do to solve this issue

Ngaicode commented 1 year ago

i'm getting the same issue

port19x commented 1 year ago

same issue is there anything we can do to solve this issue

Not right now. When a pull request with a fix becomes available I will link it here for testing

creeperkafasi commented 1 year ago

I couldn't even search for anime, seems like allanime now sends a request to api.allanime.day instead of the .to domain. The Referer header needs to stay the same though.

Changed the api.${allanime_base} to api.allanime.day in lines 168, 188 and 195 and search works Selecting an episode still doesn't work though

emanueljg commented 1 year ago

Having the same issue, ani-cli version 4.4.0

MMMNB44 commented 1 year ago

Same issue on windows 11 version 4.4.0 and 4.4.5 sadly.

woolw commented 1 year ago

yep, can't even find any anime

ani-cli version 4.4.5 Arch Linux

creeperkafasi commented 1 year ago

The person behind animdl seems to have figured it out.

This is the function they use for decrypting the new sources

And there seem to be 2 different passwords for # and ## links

I have no idea on how to implement this with bash though.

CoolnsX commented 1 year ago

It's folk, a single guy, please atleast give him credit.

trueauracoral commented 1 year ago

The mastermind behind anime web scraping. Python daddy.

Veroniclover commented 1 year ago

The folks over at animdl seem to have figured it out.

This is the function they use for decrypting the new sources

And there seem to be 2 different passwords for # and ## links

I have no idea on how to implement this with bash though.

I have created one on bash but it is too slow... Here

CoolnsX commented 1 year ago

The folks over at animdl seem to have figured it out. This is the function they use for decrypting the new sources And there seem to be 2 different passwords for # and ## links I have no idea on how to implement this with bash though.

I have created one on bash but it is too slow... Here

also It won't work in ani-cli as it follows POSIX standard, not bashisms

ryan-c-dev commented 1 year ago

Far from my knowledge to throw out serious suggestions but could it be possible to make the script POSIX compliant and faster using a shell like Dash?

Maybe impossible w/ reliance on bashisms though?

white-spaced commented 1 year ago

Far from my knowledge to throw out serious suggestions but could it be possible to make the script POSIX compliant and faster using a shell like Dash?

Maybe impossible w/ reliance on bashisms though?

it is posix compliant and it uses /bin/sh, i use dash and it works perfectly fine.

Maritsu commented 1 year ago

4.4.5 on Arch Linux; no results found

ryan-c-dev commented 1 year ago

4.4.5 on Arch Linux; no results found

it's part of the AUR, not a standard arch linux package (i.e. Pacman -Ss) , no point installing right now though as the script needs to be reworked apparently.

CoolnsX commented 1 year ago

it's part of the AUR, not a standard arch linux package (i.e. Pacman -Ss) , no point installing right now though as the script needs to be reworked apparently.

he meant that his current version is not working, you are welcome

CoolnsX commented 1 year ago

Guys there is PR opened please be patient as we test it thoroughly, we are not merging due to 2 reasons :-

  1. We all have jobs and are very busy people
  2. What if allanime changed the decryption algorithm again as we have gone through that situation when we were scraping gogoanime and they changed their keys more frequently(like everyday), so we don't that state where we merge it and allanime change their encryption/password
CoolnsX commented 1 year ago

It will be merged as soon as we get time.

HonkHonkGoose228 commented 1 year ago

4.4.5 on Arch Linux; no results found

Same here. Using Linux Mint 21.1 Vera.

justchokingaround commented 1 year ago

it doesn't matter what system you use, the script is broken atm. in the meantime you can use the fix like this, before we merge it into master:

sh <(curl -s "https://raw.githubusercontent.com/pystardust/ani-cli/hex_decryption/ani-cli")
HonkHonkGoose228 commented 1 year ago

it doesn't matter what system you use, the script is broken atm. in the meantime you can use the fix like this, before we merge it into master:

sh <(curl -s "https://raw.githubusercontent.com/pystardust/ani-cli/hex_decryption/ani-cli")

Thank you so much!

CoolnsX commented 1 year ago

fixed in #1171