--- loncom/auth/lonlogout.pm	2021/01/04 03:49:52	1.45.2.8
+++ loncom/auth/lonlogout.pm	2017/11/30 03:35:08	1.52
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Logout Handler
 #
-# $Id: lonlogout.pm,v 1.45.2.8 2021/01/04 03:49:52 raeburn Exp $
+# $Id: lonlogout.pm,v 1.52 2017/11/30 03:35:08 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -71,40 +71,45 @@ sub handler {
     my @profile;
     my $lonidsdir=$r->dir_config('lonIDsDir');
     &Apache::lonnet::transfer_profile_to_env($lonidsdir,$handle);
-    if (unlink("$lonidsdir/$handle.id")) {
-        if (($env{'user.linkedenv'} =~ /^[a-f0-9]+_linked$/) &&
-            (-l "$lonidsdir/$env{'user.linkedenv'}.id") &&
-            (readlink("$lonidsdir/$env{'user.linkedenv'}.id") eq "$lonidsdir/$handle.id")) {
-            unlink("$lonidsdir/$env{'user.linkedenv'}.id");
+    unlink("$lonidsdir/$handle.id");
+    if ($env{'user.linkedenv'} ne '') {
+        my $lonhost = $r->dir_config('lonHostID');
+        if ((-l $env{'user.linkedenv'}) && 
+            (readlink($env{'user.linkedenv'}) eq "$lonidsdir/$handle.id")) {
+            unlink($env{'user.linkedenv'});
         }
     }
     if (!$Apache::lonlocal::lh) {
         &Apache::lonlocal::get_language_handle($r);
     }
     my %temp=('logout' => time);
-    my $ip = &Apache::lonnet::get_requestor_ip();
     &Apache::lonnet::put('email_status',\%temp);
     &Apache::lonnet::log($env{'user.domain'},
 			 $env{'user.name'},
 			 $env{'user.home'},
-			 "Logout $ip");
+			 "Logout $ENV{'REMOTE_ADDR'}");
 
     &Apache::loncommon::content_type($r,'text/html');
 
-    #expire the cookies
-    my %cookies=CGI::Cookie->parse($r->header_in('Cookie'));
-    foreach my $name (keys(%cookies)) {
-        next unless ($name =~ /^lon(|S|Link|Pub)ID$/);
-        my $c = new CGI::Cookie(-name    => $name,
-                                -value   => '',
-                                -expires => '-10y',);
-        $r->headers_out->add('Set-cookie' => $c);
+    #expire the cookie
+    my $name = 'lonID';
+    if (($env{'user.name'} eq 'public') && ($env{'user.domain'} eq 'public')) {
+        $name = 'lonPubID';
+    }
+    my $c = new CGI::Cookie(-name    => $name,
+			    -value   => '',
+			    -expires => '-10y',);
+    $r->headers_out->add('Set-cookie' => $c);
+    if (($name eq 'lonID') && ($env{'user.linkedenv'})) {
+        my $other = new CGI::Cookie(-name    => 'lonLinkID',
+                                    -value   => '',
+                                    -expires => '-10y',);
+        $r->headers_out->add('Set-cookie' => $other);
     }
     $r->send_http_header;
     return OK if $r->header_only;
 # -------------------------------------------------------- Menu script and info
 
-    my $windowinfo=&Apache::lonmenu::close();
 # ---------------------------------------------------------------- Get handover
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['handover']);
     my $switch='';
@@ -125,7 +130,7 @@ sub handler {
                     $ssofile = '/'.&Apache::loncommon::clean_path($r->dir_config("lonSSOUserLogoutHeadFile_$domain"));
                     if ($ssofile eq $r->dir_config("lonSSOUserLogoutHeadFile_$domain")) {
                         if ($ssofile =~ /^\Q$londocroot\E/) {
-                            if (open(my $fh,'<',$ssofile)) {
+                            if (open(my $fh,"<$ssofile")) {
                                 $headextra = join('',<$fh>);
                                 close($fh);
                             }
@@ -137,7 +142,7 @@ sub handler {
                 $ssofile = '/'.&Apache::loncommon::clean_path($r->dir_config('lonSSOUserLogoutHeadFile'));
                 if ($ssofile eq $r->dir_config('lonSSOUserLogoutHeadFile')) {
                     if ($ssofile =~ /^\Q$londocroot\E/) {
-                        if (open(my $fh,'<',$ssofile)) {
+                        if (open(my $fh,"<$ssofile")) {
                             $headextra.= join('',<$fh>);
                             close($fh);
                         }
@@ -146,8 +151,7 @@ sub handler {
             }
         }
 	$start_page=&Apache::loncommon::start_page('Logged Out',$headextra,
-                           {'no_inline_link' => 1,
-                            'no_nav_bar'     => 1,});
+                           {'no_nav_bar'     => 1,});
 	
         $relogmessage.='<h1>'.&mt('Goodbye').'</h1>'
                       .'<p>'.&mt('Thank you for using LON-CAPA.').'</p>';
@@ -206,14 +210,14 @@ Due to security reasons in new web brows
         }
         if ($env{'request.sso.login'}
 	    && defined($r->dir_config("lonSSOUserLogoutMessageFile_$domain"))) {
-	    if (open(my $fh,'<',$r->dir_config("lonSSOUserLogoutMessageFile_$domain"))) {
+	    if (open(my $fh,$r->dir_config("lonSSOUserLogoutMessageFile_$domain"))) {
 	        $relogmessage.= join('',<$fh>);
                 close($fh);
             }
 	}
 	if ($env{'request.sso.login'}
 	    && defined($r->dir_config('lonSSOUserLogoutMessageFile'))) {
-	    if (open(my $fh,'<',$r->dir_config('lonSSOUserLogoutMessageFile'))) {
+	    if (open(my $fh,$r->dir_config('lonSSOUserLogoutMessageFile'))) {
 	        $relogmessage.= join('',<$fh>);
                 close($fh);
             }
@@ -223,17 +227,9 @@ Due to security reasons in new web brows
 # --------------------------------------------------------------- Screen Output
     $r->print(<<ENDDOCUMENT);
 $start_page
-$windowinfo
 $relogmessage
 $end_page
 ENDDOCUMENT
-    if ($env{'request.balancercookie'}) {
-        my ($balancer,$cookie) = split(/:/,$env{'request.balancercookie'});
-        if ((&Apache::lonnet::hostname($balancer)) && ($cookie =~ /^[a-f0-9]{32}$/)) {
-            $cookie = $env{'user.domain'}.'_'.$env{'user.name'}.'_'.$cookie;
-            &Apache::lonnet::delbalcookie($cookie,$balancer);
-        }
-    }
     $r->register_cleanup(\&flush_course_logs);
     return OK; 
 }