--- loncom/auth/lonacc.pm	2002/01/04 14:57:14	1.26
+++ loncom/auth/lonacc.pm	2002/10/25 13:59:40	1.36
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Cookie Based Access Handler
 #
-# $Id: lonacc.pm,v 1.26 2002/01/04 14:57:14 www Exp $
+# $Id: lonacc.pm,v 1.36 2002/10/25 13:59:40 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -34,7 +34,7 @@
 # 01/06/01,05/28,8/11,9/26,11/29 Gerd Kortemeyer
 # 12/15 Scott Harrison
 # YEAR=2002
-# 1/4 Gerd Kortemeyer
+# 1/4,2/25 Gerd Kortemeyer
 #
 ###
 
@@ -100,7 +100,7 @@ sub handler {
                $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C",hex($1))/eg;
                $name  =~ tr/+/ /;
                $name  =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C",hex($1))/eg;
-               $ENV{"form.$name"}=$value;
+	       &Apache::loncommon::add_to_env("form.$name",$value);
             }
         } else {
 	    my $contentsep=$1;
@@ -120,7 +120,7 @@ sub handler {
                         } else {
                             $value=~s/\s+$//s;
                         }
-                        $ENV{"form.$name"}=$value;
+			&Apache::loncommon::add_to_env("form.$name",$value);
                     }
                     if ($i<$#lines) {
 			$i++;
@@ -155,7 +155,7 @@ sub handler {
 
 # ---------------------------------------------------------------- Check access
 
-            if ($requrl!~/^\/adm\//) {
+            if ($requrl!~/^\/adm|public\//) {
 		my $access=&Apache::lonnet::allowed('bre',$requrl);
                 if ($access eq '1') {
 		   $ENV{'user.error.msg'}="$requrl:bre:0:0:Choose Course";
@@ -172,7 +172,27 @@ sub handler {
             $requrl=~/\.(\w+)$/;
             if (&Apache::loncommon::fileembstyle($1) eq 'ssi') {
 # ------------------------------------- This is serious stuff, get symb and log
-		my $symb=&Apache::lonnet::symbread;
+		my $query=$r->args;
+                my $symb;
+                if ($query) {
+		    &Apache::loncommon::get_unprocessed_cgi($query,['symb']);
+                }
+                if ($ENV{'form.symb'}) {
+		    $symb=&Apache::lonnet::symbclean($ENV{'form.symb'});
+                    if (&Apache::lonnet::symbverify($symb,$requrl)) {
+                      my ($map,$mid,$murl)=split(/\_\_\_/,$symb);
+                      &Apache::lonnet::symblist($map,$murl => $mid,
+                                               'last_known' => $murl);
+		    } else {
+			$r->log_reason('Invalid symb for '.$requrl.': '.
+                                       $symb);
+		        $ENV{'user.error.msg'}=
+                                "$requrl:bre:1:1:Invalid Access";
+  	                return HTTP_NOT_ACCEPTABLE; 
+                    }
+                } else {
+	            $symb=&Apache::lonnet::symbread;
+                }
                 $ENV{'request.symb'}=$symb;
                 &Apache::lonnet::courseacclog($symb);
             } else {
@@ -187,7 +207,8 @@ sub handler {
     }
 
 # -------------------------------------------- See if this is a public resource
-    if (&Apache::lonnet::metadata($requrl,'copyright') eq 'public') {
+    if (($requrl=~/^\/public\//) || 
+        (&Apache::lonnet::metadata($requrl,'copyright') eq 'public')) {
         &Apache::lonnet::logthis('Granting public access: '.$requrl);
 	$ENV{'user.name'}='public';
         $ENV{'user.domain'}='public';
@@ -196,10 +217,17 @@ sub handler {
         $ENV{'request.filename'} = $r->filename;
         return OK;
     }
-# ----------------------------------------------- Store where they wanted to go
-    
-    $ENV{'request.firsturl'}=$requrl;
-    return FORBIDDEN;
+# -------------------------------------------------------------- Not authorized
+    $requrl=~/\.(\w+)$/;
+    if ((&Apache::loncommon::fileembstyle($1) eq 'ssi') ||
+        ($requrl=~/^\/adm\/(roles|logout)/)) {
+# -------------------------- Store where they wanted to go and get login screen
+       $ENV{'request.firsturl'}=$requrl;
+       return FORBIDDEN;
+   } else {
+# --------------------------------------------------------------------- Goodbye
+       return HTTP_BAD_REQUEST;
+   }
 }
 
 1;