--- loncom/auth/lonacc.pm	2021/12/30 04:47:38	1.159.2.21.2.1
+++ loncom/auth/lonacc.pm	2021/10/26 14:17:21	1.198
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Cookie Based Access Handler
 #
-# $Id: lonacc.pm,v 1.159.2.21.2.1 2021/12/30 04:47:38 raeburn Exp $
+# $Id: lonacc.pm,v 1.198 2021/10/26 14:17:21 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -160,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|m)$/i) {
                         $value=~s/[\r\n]$//;
                     }
                     if (ref($fields) eq 'ARRAY') {
@@ -204,6 +203,14 @@ sub get_posted_cgi {
 			$fname='';
 			$fmime='';
 		    }
+                    if ($i<$#lines && $lines[$i+1]=~/^Content\-Type\:\s*([\w\-\/]+)/i) {
+                        # TODO: something with $1 !
+                        $i++;
+                    }
+                    if ($i<$#lines && $lines[$i+1]=~/^Content\-transfer\-encoding\:\s*([\w\-\/]+)/i) {
+                        # TODO: something with $1 !
+                        $i++;
+                    }
 		    $i++;
 		}
 	    } else {
@@ -298,9 +305,7 @@ sub sso_login {
     my $query = $r->args;
     my %form;
     if ($query) {
-
-        my @items = ('role','symb','iptoken','origurl','ttoken',
-                     'ltoken','linkkey','logtoken','sso');
+        my @items = ('role','symb','iptoken','origurl','ltoken','linkkey');
         &Apache::loncommon::get_unprocessed_cgi($query,\@items);
         foreach my $item (@items) {
             if (defined($env{'form.'.$item})) {
@@ -319,97 +324,12 @@ sub sso_login {
     }
 
     my ($linkprot,$linkkey);
-
-#
-# 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, or linkprot or linkkey (deep-link access).
-#
-# If Shibboleth auth is in use, but a dual log-in page is not in use,
-# and the originally requested URL was /tiny/$domain/$id (i.e.,
-# for deeplinking), then the query string will contain the sso 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, linkprot or linkkey (deep-link access).
-#
-# Otherwise the query string may contain role and symb, or if the
-# originally requested URL was /tiny/$domain/$id (i.e. for deeplinking)
-# then the query string may contain a ttoken item with a value set
-# to the name of a .tmp file in /home/httpd/perl/tmp containing either
-# linkprot or linkkey (deep-link access).
-#
-# If deep-linked, i.e., the originally requested URL was /tiny/$domain/$id
-# the linkkey may have originally been sent in POSTed data, which will
-# have been processed in lontrans.pm
-#
-
-    if ($form{'ttoken'}) {
-        my %info = &Apache::lonnet::tmpget($form{'ttoken'});
-        &Apache::lonnet::tmpdel($form{'ttoken'});
-        if ($info{'origurl'}) {
-            $form{'origurl'} = $info{'origurl'};
-        }
-        if ($info{'linkprot'}) {
-            $linkprot = $info{'linkprot'};
-        } elsif ($info{'linkkey'} ne '') {
-            $linkkey = $info{'linkkey'};
-        }
-    } elsif ($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 ($firsturl =~ m{^/tiny/$match_domain/\w+$}) {
-                $form{'origurl'} = $firsturl;
-            }
-            if ($form{'linkprot'}) {
-                $linkprot = $form{'linkprot'};
-            } elsif ($form{'linkkey'} ne '') {
-                $linkkey = $form{'linkkey'};
-            }
-            if ($form{'iptoken'}) {
-                %sessiondata = &Apache::lonnet::tmpget($form{'iptoken'});
-                my $delete = &Apache::lonnet::tmpdel($form{'iptoken'});
-            }
-        }
-    } elsif ($form{'sso'}) {
-        my $lonhost = $r->dir_config('lonHostID');
-        my $info = &Apache::lonnet::reply('tmpget:'.$form{'sso'},$lonhost);
-        &Apache::lonnet::tmpdel($form{'sso'});
-        unless (($info=~/^error/) || ($info eq 'con_lost') ||
-                ($info eq 'no_such_host')) {
-            my ($firsturl,@rest)=split(/\&/,$info);
-            if ($firsturl ne '') {
-                $form{'origurl'} = &unescape($firsturl);
-            }
-            foreach my $item (@rest) {
-                my ($key,$value) = split(/=/,$item);
-                $form{$key} = &unescape($value);
-            }
-            if ($form{'linkprot'}) {
-                $linkprot = $form{'linkprot'};
-            } elsif ($form{'linkkey'} ne '') {
-                $linkkey = $form{'linkkey'};
-            }
-        }
-    } elsif ($form{'ltoken'}) {
+    if ($form{'ltoken'}) {
         my %link_info = &Apache::lonnet::tmpget($form{'ltoken'});
         $linkprot = $link_info{'linkprot'};
         my $delete = &Apache::lonnet::tmpdel($form{'ltoken'});
-        delete($form{'ltoken'});
-    } elsif ($form{'linkkey'} ne '') {
+    }
+    if ($form{'linkkey'} ne '') {
         $linkkey = $form{'linkkey'};
     }
 
@@ -488,7 +408,7 @@ 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 = &Apache::lonnet::get_requestor_ip($r);
 	    my %info=('ip'        => $ip,
 		      'domain'    => $domain,
 		      'username'  => $user,
@@ -498,8 +418,6 @@ sub sso_login {
             foreach my $item ('role','symb','iptoken','origurl') {
                 if (exists($form{$item})) {
                     $info{$item} = $form{$item};
-                } elsif ($sessiondata{$item} ne '') {
-                    $info{$item} = $sessiondata{$item};
                 }
             }
             unless (($info{'symb'}) || ($info{'origurl'})) {
@@ -556,8 +474,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, or linkprot or linkkey for use after account
-# creation
                 $r->set_handlers('PerlHandler'=> [\&Apache::createaccount::handler]);
                 $r->handler('perl-script');
             } else {
@@ -655,7 +571,8 @@ sub handler {
             }
         } elsif ($env{'request.course.id'} &&
                  (($requrl =~ m{^/adm/$match_domain/$match_username/aboutme$}) ||
-                  ($requrl =~ m{^/public/$cdom/$cnum/syllabus$}))) {
+                  ($requrl eq "/public/$cdom/$cnum/syllabus") ||
+                  ($requrl =~ m{^/adm/$cdom/$cnum/\d+/ext\.tool$}))) {
             my $query = $r->args;
             if ($query) {
                 foreach my $pair (split(/&/,$query)) {
@@ -677,6 +594,7 @@ sub handler {
             my $lonhost = &Apache::lonnet::host_from_dns($hostname);
             if ($lonhost) {
                 my $actual = &Apache::lonnet::absolute_url($hostname,1,1);
+                my $exphostname = &Apache::lonnet::hostname($lonhost);
                 my $expected = $Apache::lonnet::protocol{$lonhost}.'://'.$hostname;
                 unless ($actual eq $expected) {
                     $env{'request.use_absolute'} = $expected;
@@ -812,8 +730,7 @@ sub handler {
                         }
                     }
                 }
-                my $clientip = &Apache::lonnet::get_requestor_ip($r);
-                $access=&Apache::lonnet::allowed('bre',$requrl,'','',$clientip,'','',$nodeeplinkcheck);
+                $access=&Apache::lonnet::allowed('bre',$requrl,'','','','','',$nodeeplinkcheck);
             }
         }
         if ($check_block) {
@@ -826,7 +743,7 @@ sub handler {
                 &Apache::blockedaccess::setup_handler($r);
                 return OK;
             }
-        } elsif ($check_access) { 
+        } elsif ($check_access) {
             if ($handle eq '') {
                 unless ($access eq 'F') {
                     if ($requrl =~ m{^/res/$match_domain/$match_username/}) {
@@ -1026,8 +943,8 @@ sub handler {
                                 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]);
+			    &Apache::lonnet::symblist($map,$murl =>[$murl,$mid],
+						      'last_known' =>[$murl,$mid]);
                         }
 		    }
 		}
@@ -1080,7 +997,7 @@ 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 $access=&Apache::lonnet::allowed('bre',$requrl,undef,undef,$clientip);
 	if ($access eq 'A') {
 	    &Apache::restrictedaccess::setup_handler($r);
 	    return OK;