You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@httpd.apache.org by "Roy T. Fielding" <fi...@kiwi.ics.uci.edu> on 1997/10/05 11:30:16 UTC

Re: [PATCH] vhosts revamp

+1 in principle, but I can't test vhost stuff.  It sure would be nice
to have it in one location.

>+const char *set_name_virtual_host (cmd_parms *cmd, void *dummy, char *arg)
>+{
>+    /* XXX: should probably be main_server->port ... but that's inaccessible 
>*/
>+    get_addresses(cmd->pool, arg, &name_vhost_list_tail, DEFAULT_PORT);
>+    return NULL;
>+}

Isn't server_conf in http_main.c accessible?  I'm not sure if it is set
at this point in the config, though.

....Roy

Re: [PATCH] vhosts revamp

Posted by Dean Gaudet <dg...@arctic.org>.

server_conf isn't accessible ... but cmd->server->port is accessible. 
Fixed. 

Dean

On Sun, 5 Oct 1997, Roy T. Fielding wrote:

> +1 in principle, but I can't test vhost stuff.  It sure would be nice
> to have it in one location.
> 
> >+const char *set_name_virtual_host (cmd_parms *cmd, void *dummy, char *arg)
> >+{
> >+    /* XXX: should probably be main_server->port ... but that's inaccessible 
> >*/
> >+    get_addresses(cmd->pool, arg, &name_vhost_list_tail, DEFAULT_PORT);
> >+    return NULL;
> >+}
> 
> Isn't server_conf in http_main.c accessible?  I'm not sure if it is set
> at this point in the config, though.
> 
> ....Roy
>