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

[Question] Issues with DSF file streaming #462

Closed
GioF71 opened this issue May 10, 2024 · 4 comments
Closed

[Question] Issues with DSF file streaming #462

GioF71 opened this issue May 10, 2024 · 4 comments
Milestone

Comments

@GioF71
Copy link

GioF71 commented May 10, 2024

Hello, and once again thank you for the great work.
I am using the new feature you implemented, dsf support.
For some reasons, I cannot properly stream a dsf file to mpd using my subsonic plugin for upmpdcli.
The issue consists in the fact that mpd converts the stream to pcm for reasons I have not determined yet. The same does not happen when streaming from MinimServer.
The subsonic plugin just serves an url to mpd: infact, if I feed the lms stream url to mpd directly using mpc (the command line client for mpd), the same problem happens, and in this case upmpdcli and the plugin are out out the scenario.

Checking the http headers of the url from MinimServer and comparing with the one from LMS, I see two main differences.

Content-Type is "audio/dsd" for LMS, while is "audio/x-dsf" for MinimServer. This looks more appropriate as it specifies more explicitly the format.
Also, in minimserver, the header "Connection" is set to keep-alive. This one is missing in lms.

Do you see something wrong here, or something that can be done in order to fix the issue?

As a side note, I can tell you that mpd complains with a "ffmpeg/dsf: Estimating duration from bitrate, this may be inaccurate" message when I play those dsf files, however it does not complain similarly for any other file (flac namely), because in any case the duration information is set by the plugin.

Thank you

epoupon added a commit that referenced this issue May 11, 2024
@epoupon
Copy link
Owner

epoupon commented May 11, 2024

Hello!
Thanks for reporting, could you please check with image 02a0d4a5 ? In this, I changed the mime type accordingly.

@GioF71
Copy link
Author

GioF71 commented May 11, 2024

Hello and thank you, I will probably in a few hours and report back to you.

@GioF71
Copy link
Author

GioF71 commented May 11, 2024

Hello, I just tried, that worked perfectly! Amazing work!!!

@epoupon
Copy link
Owner

epoupon commented May 11, 2024

Thanks for testing!

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