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

Tautulli scrobbling not working after update #77

Closed
rumpff opened this issue May 23, 2023 · 9 comments
Closed

Tautulli scrobbling not working after update #77

rumpff opened this issue May 23, 2023 · 9 comments

Comments

@rumpff
Copy link

rumpff commented May 23, 2023

I updated multiscrobbler and tautulli to the newest versions but now i get the following error in the ms console whenever a track is listened

TypeError: Cannot destructure property 'artist_name' of 'obj.body' as it is undefined.
at Function.formatPlayObj (C:\Users\Administrator\Documents\multi-scrobbler\src\sources\TautulliSource.js:16:17)
at C:\Users\Administrator\Documents\multi-scrobbler\src\index.js:210:57
at wrappedMiddleware (C:\Users\Administrator\Documents\multi-scrobbler\node_modules@awaitjs\express\index.js:116:13)
at Layer.handle [as handle_request] (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\layer.js:95:5)
at next (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\route.js:144:13)
at Route.dispatch (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\route.js:114:3)
at Layer.handle [as handle_request] (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\layer.js:95:5)
at C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\index.js:284:15
at Function.process_params (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\index.js:346:12)
at next (C:\Users\Administrator\Documents\multi-scrobbler\node_modules\express\lib\router\index.js:280:10)

and this error in tautulli

Tautulli NotificationHandler :: Notification thread exception: decoding str is not supported
Traceback (most recent call last):
File "plexpy\request.py", line 178, in request_response2
File "requests\models.py", line 1021, in raise_for_status
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://localhost:9078/tautulli

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "plexpy\notification_handler.py", line 78, in process_queue
File "plexpy\notification_handler.py", line 430, in notify
File "plexpy\notifiers.py", line 713, in send_notification
File "plexpy\notifiers.py", line 970, in notify
File "plexpy\notifiers.py", line 4228, in agent_notify
File "plexpy\notifiers.py", line 977, in make_request
File "plexpy\request.py", line 207, in request_response2
File "plexpy\request.py", line 318, in server_message
TypeError: decoding str is not supported

I updated the webhook JSON data to what's listed in the docs currently
running on windows server 2019 version 1809

i have a tautulli users with username and email, both don't work

@FoxxMD
Copy link
Owner

FoxxMD commented May 24, 2023

I've added a fix to the develop branch, please update and let me know if that fixes the issue for you.

@rumpff
Copy link
Author

rumpff commented May 24, 2023

hasn't fixed it, getting the same errors

@FoxxMD
Copy link
Owner

FoxxMD commented May 24, 2023

Can you confirm that you checked out the develop branch and pulled the latest changes? What is the output of this command when run from the project directory?

git rev-parse HEAD

@rumpff
Copy link
Author

rumpff commented May 24, 2023

Can you confirm that you checked out the develop branch and pulled the latest changes? What is the output of this command when run from the project directory?

git rev-parse HEAD

c10d46b

@FoxxMD
Copy link
Owner

FoxxMD commented May 24, 2023

What version of Tautulli are you running? It's on the settings -> Help & Info tab.

@rumpff
Copy link
Author

rumpff commented May 24, 2023

Version v2.12.3

FoxxMD added a commit that referenced this issue May 25, 2023
* Log warning if payload is empty
* Destructure request body with default empty object
@FoxxMD
Copy link
Owner

FoxxMD commented May 25, 2023

After updating the branch did you rebuild the project with npm run build or tsc -p . ?

I've added additional checks to develop if you want to try updating.

I was able to reproduce your bug report but
c10d46b should have fixed it, the TautulliSource.formatPlayObj method is only used in two places so the bug's scope was pretty small. I'm trying to remove any assumptions about why it's still occurring for you before digging any deeper (there isn't much deeper to dig).

@FoxxMD
Copy link
Owner

FoxxMD commented Jul 18, 2023

Any update @rumpff ?

@FoxxMD
Copy link
Owner

FoxxMD commented Aug 8, 2023

If you would like to revist this i can re-open.

@FoxxMD FoxxMD closed this as not planned Won't fix, can't repro, duplicate, stale Aug 8, 2023
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