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

LastFM Error - Login: User required to be logged in #105

Closed
FozzieHi opened this issue Nov 11, 2023 · 4 comments
Closed

LastFM Error - Login: User required to be logged in #105

FozzieHi opened this issue Nov 11, 2023 · 4 comments

Comments

@FozzieHi
Copy link

Describe the bug
I get an error when trying to scrobble to LastFM. I have authenticated via the callbackUri successfully (i.e. I get an OK after following the URL), but the LastFM client has a status of "Idle" and cannot send any scrobbles.

The following is my client config (some things redacted with x):

{
  "name": "LastFM",
  "type": "lastfm",
  "data": {
    "apiKey": "x",
    "secret": "x",
    "redirectUri": "x/lastfm/callback"
  }
}

Note: The example LastFM config seemed to be missing a type property, so I added it above.

Logs

error   : [Scrobblers] [Lastfm - LastFM] Login: User required to be logged in
Error: 
    at CWD/node_modules/lastfm-node-client/lib/ApiRequest.js:136:11
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async d.callApi (CWD/build/server.js:1:15592)
    at async u.refreshScrobbles (CWD/build/server.js:1:48956)
    at async u.doProcessing (CWD/build/server.js:1:42594)
    at async u.startScrobbling (CWD/build/server.js:1:39715)
    at async u.initScrobbleMonitoring (CWD/build/server.js:1:39243)

Versions (please complete the following information):
Provide version information for any related sources/clients.

  • multi-scrobbler: latest tag on Docker Hub, is there a way to get the version within the app?
@FozzieHi
Copy link
Author

I did some searching and it seems like this is similar to an issue fixed in nrubin29/finale#25. Disabling Hide recent listening information in my LastFM settings fixes the issue for me. The solution in the linked PR is to send the session key when calling a particular endpoint, do we need to do the same here to fix it?

FoxxMD added a commit that referenced this issue Nov 13, 2023
…ng session key #105

Thanks @FozzieHi for doing the legwork on finding the solution for this

Closes #105
@FoxxMD
Copy link
Owner

FoxxMD commented Nov 13, 2023

Thanks for the hint on the fix for this. You can test the fix in the docker image foxxmd/multi-scrobbler:develop

@FozzieHi
Copy link
Author

Thanks for the hint on the fix for this. You can test the fix in the docker image foxxmd/multi-scrobbler:develop

Thanks, that has fixed it. Also thanks for adding the version info to the UI :)

@FoxxMD
Copy link
Owner

FoxxMD commented Nov 29, 2023

This should now be fixed in 0.6.2

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

2 participants