Re: URIs for accelerators

From: Adrian Chadd <adrian@dont-contact.us>
Date: Sun, 27 Jan 2008 11:35:04 +0900

Actually, this is completely wrong :)

urlDefaultPort() wants a protocol_t, not a port. The trouble here
is that http->request isn't filled out, so I can't yet use
http->request->protocol to determine whether the port number
is different from the default.

Henrik, any ideas?

Adrian

On Fri, Jan 25, 2008, Adrian Chadd wrote:
> On Wed, Jan 23, 2008, Mark Nottingham wrote:
> > That's kind of what I was thinking, but was balking at the extra
> > logic. If it's already hanging around, all the better.
>
> I just replied to the bug. This code may do the right thing:
>
> if (vport)
> snprintf(http->uri, url_sz, "%s://%s:%d%s",
> conn->port->protocol, t, port, url);
> else if (port != urlDefaultPort(conn->port->protocol))
> snprintf(http->uri, url_sz, "%s://%s:%d%s",
> conn->port->protocol, t, port, url);
> else
> snprintf(http->uri, url_sz, "%s://%s%s",
> conn->port->protocol, t, url);
>
> See if that works?
>
>
>
> Adrian
Received on Sat Jan 26 2008 - 19:24:34 MST

This archive was generated by hypermail pre-2.1.9 : Wed Jan 30 2008 - 12:00:09 MST