moehmeni/syncedlyrics

Add provider Genius.com

deedy5 opened this issue · 6 comments

Thanks for the suggestion.
It works on my machine [1] but fails the tests when querying step 1. I tested with both requests and curl_cffi. Logs showed it gets 403 status code for step 1.

[1] https://github.com/moehmeni/syncedlyrics/blob/main/syncedlyrics/providers/genius.py

Try with curl-cffi, but use impersonate
self.session = requests.Session(impersonate="chrome")

If I use curl-cffi, I get an error mentioned in #29 in local. Also, the workflow test (with impersonate) could not pass as well.

curl_cffi.requests.errors.RequestsError: Failed to perform, curl: (55) Recv failure: Connection reset by peer. See https://curl.se/libcurl/c/libcurl-errors.html first for more details.

look https://curl.se/libcurl/c/libcurl-errors.html

CURLE_SEND_ERROR (55)

Failed sending network data. 

def get_lrc()
Unclear problem. Are you sure a cookie is needed here?
Try to add logging before each line that returns None to see what happens.

I commented out the tests for Genius for now, but in action, can you please confirm it works with -p Genius?

  1. megalobiz.com site's not working anymore, it says :megalobiz.com has expired,
  2. genius tests work.