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

m3u playlist support? #456

Closed
xnoreq opened this issue Jan 4, 2014 · 5 comments
Closed

m3u playlist support? #456

xnoreq opened this issue Jan 4, 2014 · 5 comments

Comments

@xnoreq
Copy link
Contributor

xnoreq commented Jan 4, 2014

Using version 0.3.1 Windows x86-64 release.

Playing a .m3u playlist file causes this error:
Playing: path\to\test.m3u
Failed to recognize file format.

Same with m3u8. Can we have those formats supported?
A simple (non-extended) m3u file only contains a path to a file / URL per line.

Btw, what are the supported playlist formats anyway? Couldn't find that info anywhere.

@ghost
Copy link

ghost commented Jan 4, 2014

m3u should work. Can you pos the file?

@xnoreq
Copy link
Contributor Author

xnoreq commented Jan 4, 2014

Nothing special at all:

D:\Music\Artist\2010 Album\01 Track.mp3
D:\Music\Artist\2010 Album\02 Track.mp3

Same error if I change line endings to LF from CR+LF, or change the encoding from ANSI to UTF-8.

@xnoreq
Copy link
Contributor Author

xnoreq commented Jan 5, 2014

mpv can open the playlist if I run it with the --playlist option.

Why is this option needed? Can't it detect that a .m3u file is a playlist?

@divVerent
Copy link
Member

This is a security measure to prevent mpv from accessing remote urls or
local files without being told so - e.g. if you associate *.mpg with mpv in
your browser. Could do stuff like reset your router then.

Personally I would at least want to allow referring to other files in the
same directory - given it allows this for mkv too.

@ghost
Copy link

ghost commented Jan 5, 2014

What divVerent said. Playlists can do several unsafe things: poke in your local network, poke in your local filesystem (including accessing device files), and 3rd it's not sure whether all protocol types are safe.

mpv can open the playlist if I run it with the --playlist option.

In this case, the m3u file doesn't have the correct header (the first line is not #EXTM3U), so the only way to distinguish this file from any other is by file extension, which mpv normally tries to avoid to do. So it's simply not detected. The --playlist option activates a fallback to open unrecognized files as a text list of files.

Personally I would at least want to allow referring to other files in the
same directory - given it allows this for mkv too.

Might be possible - not sure.

@ghost ghost closed this as completed Jan 5, 2014
This issue was closed.
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