--- loncom/auth/lonacc.pm	2021/12/12 00:53:57	1.159.2.21
+++ loncom/auth/lonacc.pm	2014/12/01 22:52:54	1.160
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Cookie Based Access Handler
 #
-# $Id: lonacc.pm,v 1.159.2.21 2021/12/12 00:53:57 raeburn Exp $
+# $Id: lonacc.pm,v 1.160 2014/12/01 22:52:54 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -109,6 +109,7 @@ sub cleanup {
     my ($r)=@_;
     if (! $r->is_initial_req()) { return DECLINED; }
     &Apache::lonnet::save_cache();
+    &Apache::lontexconvert::jsMath_reset();
     return OK;
 }
 
@@ -159,8 +160,7 @@ sub get_posted_cgi {
                         if (length($value) == 1) {
                             $value=~s/[\r\n]$//;
                         }
-                    }
-                    if ($fname =~ /\.(xls|doc|ppt)(x|m)$/i) {
+                    } elsif ($fname =~ /\.(xls|doc|ppt)x$/i) {
                         $value=~s/[\r\n]$//;
                     }
                     if (ref($fields) eq 'ARRAY') {
@@ -282,6 +282,7 @@ sub upload_size_allowed {
 sub sso_login {
     my ($r,$handle,$username) = @_;
 
+    my $lonidsdir=$r->dir_config('lonIDsDir');
     if (($r->user eq '') || ($username ne '') || ($r->user eq 'public:public') ||
         (defined($env{'user.name'}) && (defined($env{'user.domain'}))
 	  && ($handle ne ''))) {
@@ -289,15 +290,12 @@ sub sso_login {
 	return undef;
     }
 
-    my ($user) = ($r->user =~ m/^($match_username)$/);
-    if ($user eq '') {
-        return undef;
-    }
+    my ($user) = ($r->user =~ m/([a-zA-Z0-9_\-@.]*)/);
 
     my $query = $r->args;
     my %form;
     if ($query) {
-        my @items = ('role','symb','iptoken','origurl','logtoken');
+        my @items = ('role','symb','iptoken');
         &Apache::loncommon::get_unprocessed_cgi($query,\@items);
         foreach my $item (@items) {
             if (defined($env{'form.'.$item})) {
@@ -315,38 +313,6 @@ sub sso_login {
         }
     }
 
-#
-# If Shibboleth auth is in use, and a dual SSO and non-SSO login page
-# is in use, then the query string will contain the logtoken item with
-# a value set to the name of a .tmp file in /home/httpd/perl/tmp
-# containing the url to display after authentication, and also,
-# optionally, role and symb.
-#
-# Otherwise the query string may contain role and symb.
-#
-
-    if ($form{'logtoken'}) {
-        my ($firsturl,@rest);
-        my $lonhost = $r->dir_config('lonHostID');
-        my $tmpinfo = &Apache::lonnet::reply('tmpget:'.$form{'logtoken'},$lonhost);
-        my $delete = &Apache::lonnet::tmpdel($form{'logtoken'});
-        unless (($tmpinfo=~/^error/) || ($tmpinfo eq 'con_lost') ||
-                ($tmpinfo eq 'no_such_host')) {
-            (undef,$firsturl,@rest) = split(/&/,$tmpinfo);
-            if ($firsturl ne '') {
-                $firsturl = &unescape($firsturl);
-            }
-            foreach my $item (@rest) {
-                my ($key,$value) = split(/=/,$item);
-                $form{$key} = &unescape($value);
-            }
-            if ($form{'iptoken'}) {
-                %sessiondata = &Apache::lonnet::tmpget($form{'iptoken'});
-                my $delete = &Apache::lonnet::tmpdel($form{'iptoken'});
-            }
-        }
-    }
-
     my $domain = $r->dir_config('lonSSOUserDomain');
     if ($domain eq '') {
         $domain = $r->dir_config('lonDefDomain');
@@ -363,39 +329,13 @@ sub sso_login {
         }
         unless ($hosthere) {
             ($is_balancer,$otherserver) =
-                &Apache::lonnet::check_loadbalancing($user,$domain,'login');
-            if ($is_balancer) {
-                # Check if browser sent a LON-CAPA load balancer cookie (and this is a balancer)
-                my ($found_server,$balancer_cookie) = &Apache::lonnet::check_for_balancer_cookie($r);
-                if (($found_server) && ($balancer_cookie =~ /^\Q$domain\E_\Q$user\E_/)) {
-                    $otherserver = $found_server;
-                } elsif ($otherserver eq '') {
-                    my $lowest_load;
-                    ($otherserver,undef,undef,undef,$lowest_load) = &Apache::lonnet::choose_server($domain);
-                    if ($lowest_load > 100) {
-                        $otherserver = &Apache::lonnet::spareserver($r,$lowest_load,$lowest_load,1,$domain);
-                    }
-                    if ($otherserver ne '') {
-                        my @hosts = &Apache::lonnet::current_machine_ids();
-                        if (grep(/^\Q$otherserver\E$/,@hosts)) {
-                            $hosthere = $otherserver;
-                        }
-                    }
-                }
-            }
+                &Apache::lonnet::check_loadbalancing($user,$domain);
         }
-	if (($is_balancer) && (!$hosthere)) {
+
+	if ($is_balancer) {
 	    # login but immediately go to switch server to find us a new 
 	    # machine
 	    &Apache::lonauth::success($r,$user,$domain,$home,'noredirect');
-            foreach my $item (keys(%form)) {
-                $env{'form.'.$item} = $form{$item};
-            }
-            unless (($form{'symb'}) || ($form{'origurl'})) {
-                unless (($r->uri eq '/adm/roles') || ($r->uri eq '/adm/sso')) {
-                    $env{'form.origurl'} = $r->uri;
-                }
-            }
             $env{'request.sso.login'} = 1;
             if (defined($r->dir_config("lonSSOReloginServer"))) {
                 $env{'request.sso.reloginserver'} =
@@ -410,21 +350,26 @@ sub sso_login {
 	} else {
 	    # need to login them in, so generate the need data that
 	    # migrate expects to do login
-            my $ip = &Apache::lonnet::get_requestor_ip($r);
+	    my $ip;
+	    my $c = $r->connection;
+	    eval {
+	        $ip = $c->remote_ip();
+	    };
+	    if ($@) {
+	        $ip = $c->client_ip();
+	    }
 	    my %info=('ip'        => $ip,
 		      'domain'    => $domain,
 		      'username'  => $user,
 		      'server'    => $r->dir_config('lonHostID'),
 		      'sso.login' => 1
 		      );
-            foreach my $item ('role','symb','iptoken','origurl') {
+            foreach my $item ('role','symb','iptoken') {
                 if (exists($form{$item})) {
                     $info{$item} = $form{$item};
-                } elsif ($sessiondata{$item} ne '') {
-                    $info{$item} = $sessiondata{$item};
                 }
             }
-            unless (($info{'symb'}) || ($info{'origurl'})) {
+            unless ($info{'symb'}) {
                 unless (($r->uri eq '/adm/roles') || ($r->uri eq '/adm/sso')) {
                     $info{'origurl'} = $r->uri; 
                 }
@@ -436,9 +381,6 @@ sub sso_login {
                 $info{'sso.reloginserver'} = 
                     $r->dir_config('lonSSOReloginServer'); 
             }
-            if (($is_balancer) && ($hosthere)) {
-                $info{'noloadbalance'} = $hosthere;
-            }
 	    my $token = 
 		&Apache::lonnet::tmpput(\%info,
 					$r->dir_config('lonHostID'));
@@ -466,8 +408,6 @@ sub sso_login {
             $r->subprocess_env->set('SSOUserUnknown' => $user);
             $r->subprocess_env->set('SSOUserDomain' => $domain);
             if (grep(/^sso$/,@cancreate)) {
-#FIXME - need to preserve origurl, role and symb for use after account
-# creation
                 $r->set_handlers('PerlHandler'=> [\&Apache::createaccount::handler]);
                 $r->handler('perl-script');
             } else {
@@ -507,7 +447,7 @@ sub handler {
     if ($handle eq '') {
         unless ((($requrl eq '/adm/switchserver') && (!$r->is_initial_req())) ||
                 ($requrl =~ m{^/public/$match_domain/$match_courseid/syllabus}) ||
-                ($requrl =~ m{^/adm/help/}) || ($requrl eq '/adm/sso') ||
+                ($requrl =~ m{^/adm/help/}) ||
                 ($requrl =~ m{^/res/$match_domain/$match_username/})) {
 	    $r->log_reason("Cookie not valid", $r->filename);
         }
@@ -546,7 +486,7 @@ sub handler {
                 my $preserved;
                 foreach my $pair (split(/&/,$query)) {
                     my ($name, $value) = split(/=/,$pair);
-                    unless (($name eq 'symb') || ($name eq 'usehttp')) {
+                    unless ($name eq 'symb') {
                         $preserved .= $pair.'&';
                     }
                     if (($env{'request.course.id'}) && ($name eq 'folderpath')) {
@@ -574,7 +514,6 @@ sub handler {
                         if ($value =~ /^supplemental/) {
                             $suppext = 1;
                         }
-                        last;
                     }
                 }
             }
@@ -586,7 +525,7 @@ sub handler {
             my $hostname = $r->hostname();
             my $lonhost = &Apache::lonnet::host_from_dns($hostname);
             if ($lonhost) {
-                my $actual = &Apache::lonnet::absolute_url($hostname,1,1);
+                my $actual = &Apache::lonnet::absolute_url($hostname);
                 my $expected = $Apache::lonnet::protocol{$lonhost}.'://'.$hostname;
                 unless ($actual eq $expected) {
                     $env{'request.use_absolute'} = $expected;
@@ -602,117 +541,34 @@ sub handler {
         my $checkexempt;
         if ($env{'user.loadbalexempt'} eq $r->dir_config('lonHostID')) {
             if ($env{'user.loadbalcheck.time'} + 600 > time) {
-                $checkexempt = 1;
+                $checkexempt = 1;    
             }
         }
         if ($env{'user.noloadbalance'} eq $r->dir_config('lonHostID')) {
             $checkexempt = 1;
         }
-        unless (($checkexempt) || (($requrl eq '/adm/switchserver') && (!$r->is_initial_req()))) {
+        unless ($checkexempt) {
             ($is_balancer,$otherserver) =
                 &Apache::lonnet::check_loadbalancing($env{'user.name'},
                                                      $env{'user.domain'});
-            if ($is_balancer) {
-                # Check if browser sent a LON-CAPA load balancer cookie (and this is a balancer)
-                my ($found_server,$balancer_cookie) = &Apache::lonnet::check_for_balancer_cookie($r);
-                if (($found_server) && ($balancer_cookie =~ /^\Q$env{'user.domain'}\E_\Q$env{'user.name'}\E_/)) {
-                    $otherserver = $found_server;
-                }
-                unless ($requrl eq '/adm/switchserver') {
-                    $r->set_handlers('PerlResponseHandler'=>
-                                     [\&Apache::switchserver::handler]);
-                }
-                if ($otherserver ne '') {
-                    $env{'form.otherserver'} = $otherserver;
-                }
-                unless (($env{'form.origurl'}) || ($r->uri eq '/adm/roles') ||
-                        ($r->uri eq '/adm/switchserver') || ($r->uri eq '/adm/sso')) {
-                    $env{'form.origurl'} = $r->uri;
-                }
+        }
+        if ($is_balancer) {
+            $r->set_handlers('PerlResponseHandler'=>
+                             [\&Apache::switchserver::handler]);
+            if ($otherserver ne '') {
+                $env{'form.otherserver'} = $otherserver;
+            }
+            unless (($env{'form.origurl'}) || ($r->uri eq '/adm/roles') ||
+                    ($r->uri eq '/adm/switchserver') || ($r->uri eq '/adm/sso')) {
+                $env{'form.origurl'} = $r->uri;
             }
         }
 
 # ---------------------------------------------------------------- Check access
 	my $now = time;
-        my ($check_symb,$check_access,$check_block,$access,$poss_symb);
-	if ($requrl !~ m{^/(?:adm|public|(?:prt|zip)spool)/}
+	if ($requrl !~ m{^/(?:adm|public|prtspool)/}
 	    || $requrl =~ /^\/adm\/.*\/(smppg|bulletinboard)(\?|$ )/x) {
-            $check_access = 1;
-        }
-        if ((!$check_access) && ($env{'request.course.id'})) {
-            if (($requrl eq '/adm/viewclasslist') ||
-                ($requrl =~ m{^(/adm/wrapper|)\Q/uploaded/$cdom/$cnum/docs/\E}) ||
-                ($requrl =~ m{^/adm/.*/aboutme$}) ||
-                ($requrl=~m{^/adm/coursedocs/showdoc/})) {
-                $check_block = 1;
-            }
-        }
-        if (($env{'request.course.id'}) && (!$suppext)) {
-            $requrl=~/\.(\w+)$/;
-            if ((&Apache::loncommon::fileembstyle($1) eq 'ssi') ||
-                ($requrl=~/^\/adm\/.*\/(aboutme|smppg|bulletinboard)(\?|$ )/x) ||
-                ($requrl=~/^\/adm\/wrapper\//) ||
-                ($requrl=~m|^/adm/coursedocs/showdoc/|) ||
-                ($requrl=~m|\.problem/smpedit$|) ||
-                ($requrl=~/^\/public\/.*\/syllabus$/) ||
-                ($requrl=~/^\/adm\/(viewclasslist|navmaps)$/) ||
-                ($requrl=~/^\/adm\/.*\/aboutme\/portfolio(\?|$)/)) {
-                $check_symb = 1;
-            }
-        }
-        if (($check_access) || ($check_block)) {
-            if ($check_symb) {
-                if ($env{'form.symb'}) {
-                    $poss_symb=&Apache::lonnet::symbclean($env{'form.symb'});
-                } elsif (($env{'request.course.id'}) && ($r->args ne '')) {
-                    my $query = $r->args;
-                    foreach my $pair (split(/&/,$query)) {
-                        my ($name, $value) = split(/=/,$pair);
-                        $name = &unescape($name);
-                        $value =~ tr/+/ /;
-                        $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C",hex($1))/eg;
-                        if ($name eq 'symb') {
-                            $poss_symb = &Apache::lonnet::symbclean($value);
-                            last;
-                        }
-                    }
-                }
-                if ($poss_symb) {
-                    my ($possmap,$resid,$url)=&Apache::lonnet::decode_symb($poss_symb);
-                    $url = &Apache::lonnet::clutter($url);
-                    my $toplevelmap = $env{'course.'.$env{'request.course.id'}.'.url'};
-                    unless (($url eq $requrl) && (($possmap eq $toplevelmap) ||
-                                                  (&Apache::lonnet::is_on_map($possmap)))) {
-                        undef($poss_symb);
-                    }
-                    if ($poss_symb) {
-                        if ((!$env{'request.role.adv'}) && ($env{'acc.randomout'}) &&
-                            ($env{'acc.randomout'}=~/\&\Q$poss_symb\E\&/)) {
-                            undef($poss_symb);
-                        }
-                    }
-                }
-                if ($poss_symb) {
-                    $access=&Apache::lonnet::allowed('bre',$requrl,$poss_symb);
-                } else {
-                    $access=&Apache::lonnet::allowed('bre',$requrl,'','','','',1);
-                }
-            } else {
-                my $clientip = &Apache::lonnet::get_requestor_ip($r);
-                $access=&Apache::lonnet::allowed('bre',$requrl,'','',$clientip);
-            }
-        }
-        if ($check_block) {
-            if ($access eq 'B') {
-                if ($poss_symb) {
-                    if (&Apache::lonnet::symbverify($poss_symb,$requrl)) {
-                        $env{'request.symb'} = $poss_symb;
-                    }
-                }
-                &Apache::blockedaccess::setup_handler($r);
-                return OK;
-            }
-        } elsif ($check_access) { 
+	    my $access=&Apache::lonnet::allowed('bre',$requrl);
             if ($handle eq '') {
                 unless ($access eq 'F') {
                     if ($requrl =~ m{^/res/$match_domain/$match_username/}) {
@@ -729,11 +585,6 @@ sub handler {
 		return OK;
 	    }
             if ($access eq 'B') {
-                if ($poss_symb) {
-                    if (&Apache::lonnet::symbverify($poss_symb,$requrl)) {
-                        $env{'request.symb'} = $poss_symb;
-                    }
-                }
                 &Apache::blockedaccess::setup_handler($r);
                 return OK;
             }
@@ -753,7 +604,7 @@ sub handler {
                         }
                     }
                 } elsif (($handle =~ /^publicuser_\d+$/) && (&Apache::lonnet::is_portfolio_url($requrl))) {
-                    my $clientip = &Apache::lonnet::get_requestor_ip($r);
+                    my $clientip = $r->get_remote_host();
                     if (&Apache::lonnet::allowed('bre',$requrl,undef,undef,$clientip) ne 'F') {
                         $env{'user.error.msg'}="$requrl:bre:1:1:Access Denied";
                         return HTTP_NOT_ACCEPTABLE;
@@ -784,7 +635,7 @@ sub handler {
 	    $env{'user.domain'} eq 'public' &&
 	    $requrl !~ m{^/+(res|public|uploaded)/} &&
 	    $requrl !~ m{^/adm/[^/]+/[^/]+/aboutme/portfolio$ }x &&
-            $requrl !~ m{^/adm/blockingstatus/.*$} &&
+        $requrl !~ m{^/adm/blockingstatus/.*$} &&
 	    $requrl !~ m{^/+adm/(help|logout|restrictedaccess|randomlabel\.png)}) {
 	    $env{'request.querystring'}=$r->args;
 	    $env{'request.firsturl'}=$requrl;
@@ -793,8 +644,16 @@ sub handler {
 # ------------------------------------------------------------- This is allowed
 	if ($env{'request.course.id'}) {
 	    &Apache::lonnet::countacc($requrl);
+	    $requrl=~/\.(\w+)$/;
             my $query=$r->args;
-            if ($check_symb) {
+	    if ((&Apache::loncommon::fileembstyle($1) eq 'ssi') ||
+		($requrl=~/^\/adm\/.*\/(aboutme|smppg|bulletinboard)(\?|$ )/x) ||
+		($requrl=~/^\/adm\/wrapper\//) ||
+		($requrl=~m|^/adm/coursedocs/showdoc/|) ||
+		($requrl=~m|\.problem/smpedit$|) ||
+		($requrl=~/^\/public\/.*\/syllabus$/) ||
+                ($requrl=~/^\/adm\/(viewclasslist|navmaps)$/) ||
+                ($requrl=~/^\/adm\/.*\/aboutme\/portfolio(\?|$)/)) {
 # ------------------------------------- This is serious stuff, get symb and log
 		my $symb;
 		if ($query) {
@@ -802,42 +661,17 @@ sub handler {
 		}
 		if ($env{'form.symb'}) {
 		    $symb=&Apache::lonnet::symbclean($env{'form.symb'});
-                    if (($requrl eq '/adm/navmaps') ||
-                        ($requrl =~ m{^/adm/wrapper/}) ||
-                        ($requrl =~ m{^/adm/coursedocs/showdoc/})) {
-                        unless (&Apache::lonnet::symbverify($symb,$requrl)) {
-                            if (&Apache::lonnet::is_on_map($requrl)) {
-                                $symb = &Apache::lonnet::symbread($requrl);
-                                unless (&Apache::lonnet::symbverify($symb,$requrl)) {
-                                    undef($symb);
-                                }
-                            }
-                        }
-                        if ($symb) {
-                            if ($requrl eq '/adm/navmaps') {
-                                my ($map,$mid,$murl)=&Apache::lonnet::decode_symb($symb);
-                                &Apache::lonnet::symblist($map,$murl => [$murl,$mid]);
-                            } elsif (($requrl =~ m{^/adm/wrapper/}) ||
-                                     ($requrl =~ m{^/adm/coursedocs/showdoc/})) {
-                                my ($map,$mid,$murl)=&Apache::lonnet::decode_symb($symb);
-                                if ($map =~ /\.page$/) {
-                                    my $mapsymb = &Apache::lonnet::symbread($map);
-                                    ($map,$mid,$murl)=&Apache::lonnet::decode_symb($mapsymb);
-                                }
-                                &Apache::lonnet::symblist($map,$murl => [$murl,$mid],
-                                                          'last_known' =>[$murl,$mid]);
-                            }
-                        }
+		    if ($requrl =~ m|^/adm/wrapper/|
+			|| $requrl =~ m|^/adm/coursedocs/showdoc/|) {
+			my ($map,$mid,$murl)=&Apache::lonnet::decode_symb($symb);
+			&Apache::lonnet::symblist($map,$murl => [$murl,$mid],
+						  'last_known' =>[$murl,$mid]);
 		    } elsif ((&Apache::lonnet::symbverify($symb,$requrl)) ||
 			     (($requrl=~m|(.*)/smpedit$|) &&
 			      &Apache::lonnet::symbverify($symb,$1)) ||
                              (($requrl=~m|(.*/aboutme)/portfolio$|) &&
                               &Apache::lonnet::symbverify($symb,$1))) {
 			my ($map,$mid,$murl)=&Apache::lonnet::decode_symb($symb);
-                        if (($map =~ /\.page$/) && ($requrl !~ /\.page$/)) {
-                            my $mapsymb = &Apache::lonnet::symbread($map);
-                            ($map,$mid,$murl)=&Apache::lonnet::decode_symb($mapsymb);
-                        }
 			&Apache::lonnet::symblist($map,$murl => [$murl,$mid],
 						  'last_known' =>[$murl,$mid]);
 		    } else {
@@ -851,72 +685,24 @@ sub handler {
                     if ($requrl=~m{^(/adm/.*/aboutme)/portfolio$}) {
                         $requrl = $1;
                     }
-		    $symb=&Apache::lonnet::symbread($requrl);
-                    if (&Apache::lonnet::is_on_map($requrl) && $symb) {
-                        my ($encstate,$invalidsymb);
-                        unless (&Apache::lonnet::symbverify($symb,$requrl,\$encstate)) {
-                            $invalidsymb = 1;
-                            #
-                            # If $env{'request.enc'} inconsistent with encryption expected for $symb
-                            # retrieved by lonnet::symbread(), call again to check for an instance of
-                            # $requrl in the course for which expected encryption matches request.enc.
-                            # If symb for different instance passes lonnet::symbverify(), use that as
-                            # the symb for $requrl and call &Apache::lonnet::allowed() for that symb.
-                            # Report invalid symb if there is no other symb. Redirect to /adm/ambiguous
-                            # if multiple possible symbs consistent with request.enc available for $requrl.
-                            #
-                            if (($env{'request.enc'} && !$encstate) || (!$env{'request.enc'} && $encstate)) {
-                                my %possibles;
-                                my $nocache = 1;
-                                my $oldsymb = $symb;
-                                $symb = &Apache::lonnet::symbread($requrl,'','','',\%possibles,$nocache);
-                                if (($symb) && ($symb ne $oldsymb)) {
-                                    if (&Apache::lonnet::symbverify($symb,$requrl)) {
-                                        my $access=&Apache::lonnet::allowed('bre',$requrl,$symb);
-                                        if ($access eq 'B') {
-                                            $env{'request.symb'} = $symb;
-                                            &Apache::blockedaccess::setup_handler($r);
-                                            return OK;
-                                        } elsif (($access eq '2') || ($access eq 'F')) {
-                                            $invalidsymb = '';
-                                        }
-                                    }
-                                } elsif (keys(%possibles) > 1) {
-                                    $r->internal_redirect('/adm/ambiguous');
-                                    return OK;
-                                }
-                            }
-                            if ($invalidsymb) {
-                                if ($requrl eq '/adm/navmaps') {
-                                    undef($symb);
-                                } else {
-                                    $r->log_reason('Invalid symb for '.$requrl.': '.$symb);
-                                    $env{'user.error.msg'}=
-                                        "$requrl:bre:1:1:Invalid Access";
-                                    return HTTP_NOT_ACCEPTABLE;
-                                }
-                            }
-                        }
-                    }
-		    if ($symb) {
-			my ($map,$mid,$murl)=
-			    &Apache::lonnet::decode_symb($symb);
-                        if ($requrl eq '/adm/navmaps') {
-                            &Apache::lonnet::symblist($map,$murl =>[$murl,$mid]);
-                        } else {
-                            if (($map =~ /\.page$/) && ($requrl !~ /\.page$/)) {
-                                my $mapsymb = &Apache::lonnet::symbread($map);
-                                ($map,$mid,$murl)=&Apache::lonnet::decode_symb($mapsymb);
-                            }
-                            &Apache::lonnet::symblist($map,$murl =>[$murl,$mid],
-                                                      'last_known' =>[$murl,$mid]);
-                        }
+                    unless ($suppext) {
+		        $symb=&Apache::lonnet::symbread($requrl);
+		        if (&Apache::lonnet::is_on_map($requrl) && $symb &&
+			    !&Apache::lonnet::symbverify($symb,$requrl)) {
+			    $r->log_reason('Invalid symb for '.$requrl.': '.$symb);
+			    $env{'user.error.msg'}=
+			        "$requrl:bre:1:1:Invalid Access";
+			    return HTTP_NOT_ACCEPTABLE; 
+		        }
+		        if ($symb) {
+			    my ($map,$mid,$murl)=
+			        &Apache::lonnet::decode_symb($symb);
+			    &Apache::lonnet::symblist($map,$murl =>[$murl,$mid],
+						      'last_known' =>[$murl,$mid]);
+		        }
 		    }
 		}
 		$env{'request.symb'}=$symb;
-                if (($env{'request.symbread.cached.'}) && ($env{'request.symbread.cached.'} ne $symb)) {
-                    $env{'request.symbread.cached.'} = $symb;
-                }
 		&Apache::lonnet::courseacclog($symb);
 	    } else {
 # ------------------------------------------------------- This is other content
@@ -961,8 +747,8 @@ sub handler {
     }
 # ------------------------------------ See if this is a viewable portfolio file
     if (&Apache::lonnet::is_portfolio_url($requrl)) {
-        my $clientip = &Apache::lonnet::get_requestor_ip($r);
-	my $access=&Apache::lonnet::allowed('bre',$requrl,undef,undef,$clientip);
+        my $clientip = $r->get_remote_host();
+        my $access=&Apache::lonnet::allowed('bre',$requrl,undef,undef,$clientip);
 	if ($access eq 'A') {
 	    &Apache::restrictedaccess::setup_handler($r);
 	    return OK;