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

JavaScript Error: Tried to write a message to a closed stream #318

Closed
Emaro opened this issue May 12, 2022 · 3 comments
Closed

JavaScript Error: Tried to write a message to a closed stream #318

Emaro opened this issue May 12, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@Emaro
Copy link

Emaro commented May 12, 2022

Describe the bug
Every few minutes, a JavaScript error occures: Tried to write a message to a closed stream. (see screenshot). The playback contines fine, also after clicking OK.

Edit: If you don't click the error message away, the playback stops after a while. After clicking on OK, more of the same errors show up, but if you click all away, the playback resumes.

To Reproduce

  1. Open Sonixd
  2. Play music
  3. Scrub or wait for the song to change and sometimes it happens randomly as well

Expected behavior
No JavaScript errors, more specifically no writes to closed streams.

Screenshots
image

Desktop (please complete the following information):

  • OS: Pop!_OS 22.04 LTS
  • Version: 0.15.1
  • Server: Subsonic API of the Nextcloud Music App v1.5.2

Additional context
Also, you may want to make the error message selectable, so users can copy + past it instead of capturing a screenshot. This would also allow to remove personal information more easily.

@Emaro Emaro added the bug Something isn't working label May 12, 2022
@Emaro
Copy link
Author

Emaro commented May 12, 2022

This is probably related: dbusjs/mpris-service#46

@michal4132
Copy link

It looks like "mpris:trackid" entry only allows letters and numbers. In my case arg.id contains something like "tr-991". Replacing arg.id with arg.id.replace('-','') fixes this error.

@jeffvli
Copy link
Owner

jeffvli commented Jun 1, 2022

@michal4132 Thanks for the solution, I'll add push this fix for the next version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants