hakerdefo / cmus-lyrics

Fetches lyrics of current song playing in cmus music player.
Creative Commons Zero v1.0 Universal
48 stars 10 forks source link

error: can not connect to site "makeitpersonal" #10

Open koenige50 opened 1 year ago

koenige50 commented 1 year ago

cmus-lyrics always gives me this error:

Can Not Connect To The Site "makeitpersonal"! Make Sure Your Internet Connection Is Active! If The Problem Persists You Can Report It At, GitHub Repository Of The Project By Creating, An Issue There. Here Is The Link To The Repo,

I am using it on an old Mac running Catalina with all dependencies installed and working via Homebrew. I can also access https://makeitpersonal.co with the web browser without any issue. Any idea what the problem might be?

hakerdefo commented 1 year ago

Hi there can you post me the output of the following command,

wget --spider --user-agent="Mozilla/5.0 Gecko/20100101" --no-check-certificate "https://makeitpersonal.co"

Thanks!

koenige50 commented 1 year ago

Here goes wget's output:

$ wget --spider --user-agent="Mozilla/5.0 Gecko/20100101" --no-check-certificate ["https://makeitpersonal.co"](https://makeitpersonal.co/) Spider mode enabled. Check if remote file exists.
--2023-01-10 19:34:17--  https://makeitpersonal.co/
Resolving makeitpersonal.co (makeitpersonal.co)... 107.170.105.41
Connecting to makeitpersonal.co (makeitpersonal.co)|107.170.105.41|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Remote file exists and could contain further links,
but recursion is disabled -- not retrieving.
hakerdefo commented 1 year ago

Hi there,

please open cmus-lyrics in a text editor and right after the very first line #!/usr/bin/env bash add the following line,

set -x

save the file and run the script. This will print more information in the terminal and help us debug the problem. If you are not able to figure the problem out, then paste the debug output here.

koenige50 commented 1 year ago

Below the output with set -x. Though I am no expert, it seems to me that locally everything works and the problem lies in the results returned by the remote server. Checking the files stored in /tmp/cmus_lyrics, they all contain the correct song title and artist, followed by Bots have beat this API for the time being, sorry! .

+ :
+ cmus_check
++ cmus-remote -Q
+ CMUSQ='status playing
file ~/Music/The The/The The - 1983 - Soul Mining/04 Uncertain Smile.mp3
duration 429
position 179
tag artist The The
tag album Soul Mining
tag title Uncertain Smile
tag date 1983
tag genre Rock
tag tracknumber 4
tag composer Matt Johnson
tag label Epic/Legacy
tag publisher Epic/Legacy
set aaa_mode album
set continue true
set play_library true
set play_sorted false
set replaygain disabled
set replaygain_limit true
set replaygain_preamp 0.000000
set repeat false
set repeat_current false
set shuffle off
set softvol true
set vol_left 100
set vol_right 100'
+ [[ -z status playing
file ~/Music/The The/The The - 1983 - Soul Mining/04 Uncertain Smile.mp3
duration 429
position 179
tag artist The The
tag album Soul Mining
tag title Uncertain Smile
tag date 1983
tag genre Rock
tag tracknumber 4
tag composer Matt Johnson
tag label Epic/Legacy
tag publisher Epic/Legacy
set aaa_mode album
set continue true
set play_library true
set play_sorted false
set replaygain disabled
set replaygain_limit true
set replaygain_preamp 0.000000
set repeat false
set repeat_current false
set shuffle off
set softvol true
set vol_left 100
set vol_right 100 ]]
+ cmus_status
++ cmus-remote -Q
++ grep stopped
+ CM_ST=
+ [[ -n '' ]]
+ tag_check
++ cmus-remote -Q
++ grep -m 1 title
++ cut -d ' ' -f 3-
+ TITLE='Uncertain Smile'
+ [[ -z Uncertain Smile ]]
++ cmus-remote -Q
++ grep -m 1 artist
++ cut -d ' ' -f 3-
+ ARTIST='The The'
+ [[ -z The The ]]
+ link_check_mip
+ wget --spider '--user-agent=Mozilla/5.0 Gecko/20100101' --no-check-certificate --quiet --timeout=30 -q https://makeitpersonal.co -O /dev/null
+ get_lyrics_mip
+ work_directory=/tmp/cmus_lyrics
+ test -d /tmp/cmus_lyrics
++ echo 'Uncertain Smile'
++ perl -MURI::Escape -ne 'chomp;print uri_escape($_),"\n"'
+ TITLE2=Uncertain%20Smile
++ echo 'The The'
++ perl -MURI::Escape -ne 'chomp;print uri_escape($_),"\n"'
+ ARTIST2=The%20The
++ echo -n 'Uncertain SmileThe The'
++ base32
+ FILE_HASH=KVXGGZLSORQWS3RAKNWWS3DFKRUGKICUNBSQ====
++ echo -n KVXGGZLSORQWS3RAKNWWS3DFKRUGKICUNBSQ====
++ sha256sum
++ awk '{print $1}'
+ FILE_HASH2=0e6896ecd7ba589dce92583503ebce593df60cdcb1a30f7cbfbd5c78a4a09cae
+ LYRIC_FOUND=/tmp/cmus_lyrics/0e6896ecd7ba589dce92583503ebce593df60cdcb1a30f7cbfbd5c78a4a09cae
++ echo -n KVXGGZLSORQWS3RAKNWWS3DFKRUGKICUNBSQ====
++ sha1sum
++ awk '{print $1}'
+ FILE_HASH_MISSING=119a99555eb74a6f88ca998127d3ecbb75a98176
+ LYRIC_MISSING=/tmp/cmus_lyrics/119a99555eb74a6f88ca998127d3ecbb75a98176
++ echo -n KVXGGZLSORQWS3RAKNWWS3DFKRUGKICUNBSQ====
++ sha512sum
++ awk '{print $1}'
+ FILE_HASH_MIP=96e4974b0865a992ae03d6218d9b7488f87dc8748acf25f78f5ac56ca568e0527d0ebb43380a4d69c740476b9f7cec65c6935b0e723f56499a6d73fdf833fca1
+ MIP_FILE=/tmp/cmus_lyrics/96e4974b0865a992ae03d6218d9b7488f87dc8748acf25f78f5ac56ca568e0527d0ebb43380a4d69c740476b9f7cec65c6935b0e723f56499a6d73fdf833fca1
+ '[' '!' -f /tmp/cmus_lyrics/96e4974b0865a992ae03d6218d9b7488f87dc8748acf25f78f5ac56ca568e0527d0ebb43380a4d69c740476b9f7cec65c6935b0e723f56499a6d73fdf833fca1 ']'
hakerdefo commented 1 year ago

@koenige50 Thanks for the patience and help with debugging this :+1:

I've modified the cmus-lyrics and this hopefully should fix the issue. Can you try the following version out and see if this fixes your issue or not?

cmus-lyrics-update

MagnusPGBerg commented 3 months ago

Worked for me, with the update, now then I decided to use Cmus-Lyrics. Thanks Hakerdefo.

hakerdefo commented 3 months ago

@MagnusPGBerg I'm glad it's working for you :smile: