[MPlayer-dev-eng] [PATCH 1/7] Unescape login/password before base64 encode

[Reimar Döffinger]

On Sun, Oct 24, 2010 at 09:07:50PM +0200, Clément Bœsch wrote:
> On Sun, Oct 24, 2010 at 08:31:06PM +0200, Reimar Döffinger wrote:
> > On Sun, Oct 24, 2010 at 08:10:00PM +0200, Clément Bœsch wrote:
> > > On Mon, Oct 18, 2010 at 12:05:51PM +0200, Clément Bœsch wrote:
> > > > I already sent it in a previous mail: it allows login/password to be
> > > > unescaped before base64 encode so auth with special characters now
> > > > works.
> > > 
> > > Still no news about this first patch; it should not require a lot of
> > > skills to review this stuff, even if it's 8 patches. Can anyone do it, or
> > > at least start with this one please? Those bugs are really annoying and
> > > easy to fix…
> > 
> > It's not at all easy to figure out if they actually fix more or break more...
> > This first patch for example I don't think anyone who has a %-sign in the
> > password will like.
> > Does e.g. Firefox really try to unescape the password?
> 
> The unescape is done because the URL is url-encoded in MPlayer (and so
> login/password). '%' seems to work:
> 
> % ./mplayer 'http_proxy://login_x:a%b%c@localhost:8123/http://localhost:8000
> [...]

Hm, ok, however I suspect that the whole escaping
handling in MPlayer is rather broken so I am not sure if
this is more a way of fixing things or if it's more kind of
adding a bug to hide the other bug that will make other
fixes more difficult...