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

Changing property results in EPIPE errors #46

Open
OzymandiasTheGreat opened this issue Jul 11, 2020 · 3 comments
Open

Changing property results in EPIPE errors #46

OzymandiasTheGreat opened this issue Jul 11, 2020 · 3 comments

Comments

@OzymandiasTheGreat
Copy link

What am I doing wrong? I play around with player.js example, with the addition of error handler so I can see errors in console.
I'm doing this in an electron app if that makes a difference.

I create Player object and get MPRIS notification for unknown track.
I try to change metadata or playbackStatus and it immediately throws EPIPE or write stream after closed errors.
If I don't change properties everything works, but it's kinda useless like that.

@acrisci
Copy link
Member

acrisci commented Jul 11, 2020

I've never seen this error before and I'm not sure why it would do this. Can you provide a sample so I can reproduce the issue myself?

@OzymandiasTheGreat
Copy link
Author

So I did more testing and apparently I missed dbus object path specification.
My player plays internet streams and I was using uris as mpris:trackid's, which contained invalid characters.
And that caused stream to be closed prematurely.
Not really a bug, but it'd be nice if path limitations where mentioned in the readme somewhere.

Anyway, thanks for your time and for this library, saves me a lot of time :)

@acrisci
Copy link
Member

acrisci commented Jul 12, 2020

I really thought I had validators for that to give you good error messages. I'll have to check on that.

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