Closed arcresu closed 7 years ago
Just to let people know. Upstream has fixed the issue so this doesn't need to be implemented.
While the bug in beets may have been fixed, this issue is independent of that; I was only using it as an example.
My point was that some other mpd clients could handle the malformed input since they were a little more forgiving. The mpd API seems a bit underspecified: the fact that only integers should be provided here is not documented for example. Of course this is unlikely to be an issue if you only want to support mpd itself and not unofficial implementations.
Whenever I start mpdris2, it crashes with:
It seems that mpd requires integer calues for times, but the particular server I'm using (beets bpd) has a bug (beetbox/beets#2394) where it is returning non-integers. The problem is with the server implementation, however every other mpd client software I've tried has been able to cope with the malformed response from this server.
Perhaps the handling of status fields could be a bit more robust? Changing
int()
toint(float())
would fix this particular issue, but it could be even more robust than that, and for other fields as well.