--- trunk/lib/Frey/Server.pm	2008/07/08 12:20:08	60
+++ trunk/lib/Frey/Server.pm	2008/07/09 12:08:07	66
@@ -17,16 +17,16 @@
 use vars qw( $repl $server );
 
 #$repl = Continuity::REPL->new;
-$server = Continuity->new(
-	port => 16001,
-	path_session => 1,
-	cookie_session => 'sid',
-	callback => \&main,
-	debug_level => 1,
-	staticp => sub { $_[0]->url =~ m/\.(jpg|jpeg|gif|png|css|ico|js|html?)$/ },
-);
 
 sub run {
+	$server = Continuity->new(
+		port => 16001,
+		path_session => 1,
+		cookie_session => 'sid',
+		callback => \&main,
+		debug_level => 1,
+		staticp => sub { $_[0]->url =~ m/\.(jpg|jpeg|gif|png|css|ico|js|html?)$/ },
+	);
 	$server->loop;
 }
 
@@ -42,15 +42,27 @@
 	warn $req->request->header('User_Agent');
 #warn dump( $req );
 
-	if ( $path =~ m!/~/([^/]+)(?:/([^/]*))?! ) {
-		my $f = Frey::Introspect->new( package => $1 );
-		$f->html( $req );
-	}
+	eval {
 
-	if ( $path =~ m!/ob/([^/]+)(?:/([^/]*))?! ) {
-		my $f = Frey::ObjectBrowser->new;
-		$f->html( $req );
+		if ( $path =~ m!/~/([^/]+)(?:/([^/]*))?! ) {
+			my $f = Frey::Introspect->new( package => $1 );
+			$f->html( $req );
+		}
+
+		if ( $path =~ m!/ob/([^/]+)(?:/([^/]*))?! ) {
+			my $f = Frey::ObjectBrowser->new;
+			$f->html( $req );
+		}
+
+	};
+
+	if ( $@ ) {
+		warn $@;
+		#$req->conn->send_error( 404 );	# FIXME this should probably be 500, but we can't ship page with it
+		$req->print( qq{<pre class="error">$@<pre>} );
+		$req->next;
 	}
+
 	# If this is a request for the pushtream, then give them that
 	if($path =~ /pushstream/) {
 		pushstream($req);
@@ -61,20 +73,22 @@
 		send_message($req);
 	}
 
-	while ( 1 ) {
-		my $f = Frey->new;
-		my $classes = Continuity::Widget::DomNode->create(
-			ul => [
-				map {
-					warn dump( $_ );
-					my ( $package, $path ) = %$_;
-					( li => [ a => { href => '/~/' . $package . '/' } => [ $package ], " <tt>$path</tt>" ] )
-				} @{ $f->classes }
-			],
-		)->to_string;
-		$req->print( $classes );
-		$req->next;
-	}
+	my $f = Frey->new;
+	my $classes = Continuity::Widget::DomNode->create(
+		ul => [
+			map {
+				warn dump( $_ );
+				my ( $package, $path ) = %$_;
+				( li => [
+					a => { href => '/~/' . $package . '/' } => [ $package ],
+					' ',
+					a => { href => '/ob/' . $package } => [ 'browse' ],
+					" <tt>$path</tt>"
+				] )
+			} @{ $f->classes }
+		],
+	)->to_string;
+	$req->print( $classes );
 }
 
 # Here we accept a connection to the browser, and keep it open. Meanwhile we