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

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

Open
koenige50 opened this issue Jan 9, 2023 · 7 comments
Open

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

koenige50 opened this issue Jan 9, 2023 · 7 comments

Comments

@koenige50
Copy link

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Owner

@koenige50 Thanks for the patience and help with debugging this 👍

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
Copy link

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

@hakerdefo
Copy link
Owner

@MagnusPGBerg I'm glad it's working for you 😄

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

No branches or pull requests

3 participants