[MPlayer-dev-eng] [PATCH] Add support for login/password in http_proxy env variable
ubitux at gmail.com
Fri Dec 3 20:53:48 CET 2010
On Mon, Nov 29, 2010 at 11:09:40PM +0100, Clément Bœsch wrote:
> On Sun, Nov 28, 2010 at 12:01:16AM +0100, Clément Bœsch wrote:
> > On Sat, Nov 27, 2010 at 11:33:28AM +0100, Reimar Döffinger wrote:
> > > On Wed, Nov 24, 2010 at 08:34:40PM +0100, Clément Bœsch wrote:
> > > > + if (proxy->username && proxy->password)
> > > > + return snprintf(dst, dst_size, "http_proxy://%s:%s@%s:%d/%s",
> > > > + proxy->username, proxy->password, proxy->hostname,
> > > > + proxy->port, host_url);
> > > > + else
> > > > + return snprintf(dst, dst_size, "http_proxy://%s:%d/%s",
> > > > + proxy->hostname, proxy->port, host_url);
> > >
> > > I think some other code supported also setting only username.
> > Updated.
> > > Also is port certain to be valid (!= 0) here?
> > Not it's set to 0 (this is the current behaviour) and handled later. I
> > don't really like this method but changing it belongs to another commit.
> OK to commit?
No news so I suppose it's ok; i'll commit this at the end of the week end
if there is not objection.
Not sent from a jesusPhone.
More information about the MPlayer-dev-eng