[vlc-devel] [patch] [LUA HTTP] adding file:// to the Input field
esaule at bmi.osu.edu
Fri Feb 25 02:17:08 CET 2011
> > Luckily, it happens there is a function for doing that in
> > MRL. I checked the behavior of the code in presence of spaces and
> > korean caracters.
> Nice! However encodeURI() still leaves some characters encoded. For
> example, a file name containing # will cause an error. Also, this won't
> work properly on windows: backslashes will be encoded instead of being
> changed to slashes.
usefull: encodeURI(), encodeURIcomponent() and escape(), they do not
all encode the same things. I did not like encodeURIcomponent because
escape is the one we are looking for.
Maybe we should convert the \ into / then call escape ?
> But I'm not too sure whether it's the right place to fix this. There are
> a number "VLC remote" applications, and they rely on the server-side
> If the server stops accepting mere file paths in 1.2, won't it break
> all of these applications? Do we care?
I tried to match the "input" field with the behavior of the MRL field
in the default vlc interface in media -> advanced open file -> show
more option. The interface encodes the quotes, none english
characters, spaces, but also arobase. But that's more a hack than a
fix. I am not sure what the http interface expect so I assumed any
valid MRL should be valid as well.
> > quotes leading to the wrong MRL. I removed them.
> Thanks for pointing that out. The issue was caused by a bug server-side
> preventing the unescaping of the quotes, so that's where I fixed it.
> Although the interest of escaping quotes and backslashes here is still
> dubious to me.
I am not sure there is a point in escaping quotes. The default vlc
interface actually transform quotes in %27.'
More information about the vlc-devel