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

John Peacock jpeacock at rowman.com
Fri Jul 14 20:49:48 EDT 2006


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:

=== 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


More information about the jifty-devel mailing list