[jifty-devel] Re: [Jifty-commit] r1574 - in jifty/trunk: lib/Jifty t/TestApp/t

[Jesse Vincent]

On Fri, Jul 14, 2006 at 08:49:48PM -0400, John Peacock wrote:
> jifty-commit at lists.jifty.org wrote:
> > Log:
> >  r13964 at pinglin:  jesse | 2006-07-14 19:12:07 -0400
> >  * Switched Jifty::Web::url to use uri.pm rather than "heuristics"
> 
> But this never displays the port number (even if it is 8888 like in the test
> server).  And it displays two trailing slashes.  Did you mean this instead:

Then I've done something wrong. I'll poke harder at it ;)

> === lib/Jifty/Server.pm
> ==================================================================
> --- lib/Jifty/Server.pm (revision 1270)
> +++ lib/Jifty/Server.pm (local)
> @@ -98,7 +98,7 @@
> 
>  sub print_banner {
>      my $self = shift;
> -    $self->log->info("You can connect to your server at ", Jifty::Web->url, "/");
> +    $self->log->info("You can connect to your server at ", Jifty::Web->url);
>  }
> 
>  =head2 recorder_prefix
> === lib/Jifty/Web.pm
> ==================================================================
> --- lib/Jifty/Web.pm    (revision 1270)
> +++ lib/Jifty/Web.pm    (local)
> @@ -120,7 +120,7 @@
>      if ($args{'scheme'}) {
>          $self->log->error("Jifty->web->url no longer accepts a 'scheme' argument");
>      }
> -    my $uri = URI->new($url);
> +    my $uri = URI->new("$url:$port");
> 
>      if ($ENV{'HTTP_HOST'}) {
>          return $uri->scheme ."://".$ENV{'HTTP_HOST'};
> 
> 
> John
> 
> -- 
> John Peacock
> Director of Information Research and Technology
> Rowman & Littlefield Publishing Group
> 4501 Forbes Blvd
> Suite H
> Lanham, MD 20706
> 301-459-3366 x.5010
> fax 301-429-5747
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
> 

--