X-Git-Url: https://git.treefish.org/fex.git/blobdiff_plain/7fa382617fbaccc0ce522b2b3adbbee9db5ad227..20150729:/bin/fex_cleanup

diff --git a/bin/fex_cleanup b/bin/fex_cleanup
index c54b2e0..1adec4e 100755
--- a/bin/fex_cleanup
+++ b/bin/fex_cleanup
@@ -38,7 +38,7 @@ $| = 1;
 
 # use fex.ph for site configuration!
 our ($FEXHOME);
-our ($spooldir,$logdir,$docdir);
+our ($spooldir,@logdir,$docdir);
 our ($akeydir,$ukeydir,$dkeydir,$skeydir,$gkeydir,$xkeydir,$lockdir);
 our ($durl,$debug,$autodelete,$hostname,$admin,$admin_pw,$bcc);
 $keep_default = 5;
@@ -46,6 +46,8 @@ $keep_default = 5;
 # load common code, local config : $HOME/lib/fex.ph
 require "$FEXLIB/fex.pp" or die "$0: cannot load $FEXLIB/fex.pp - $!\n";
 
+my $logdir = $logdir[0];
+
 # localized functions 
 # (needed for reminder and account reactivation e-mails)
 foreach my $lf (glob "$FEXHOME/locale/*/lib/lf.pl") { require $lf }
@@ -64,13 +66,13 @@ $today = time;
 $isodate = isodate($today);
 
 chdir $spooldir or die "$0: $spooldir - $!\n";
-open L,">>$logdir/cleanup.log";
+# open L,">>$logdir/cleanup.log";
 
 # clean up regular spool
 opendir $spooldir,'.' or die "$0: $spooldir - $!\n";
 while ($to = readdir $spooldir) {
-  next if $to !~ /@/ or -l $to;
-  if (@demo and -f "$to/.demo" and time > mtime("$to/.demo")+$demo[1]*DS) {
+  next if $to !~ /@/ or $_ = readlink($to) and not /\//;
+  if (@demo and -f "$to/.demo" and time > lmtime("$to/.demo")+$demo[1]*DS) {
     logdel($to,"demo user $to deleted");
     next;
   }
@@ -117,7 +119,7 @@ while ($to = readdir $spooldir) {
         if ($lc) {
           if (-f "$user/\@~" and not "$user/@") {
             rename "$user/\@~","$user/@" unless $opt_d;
-            logv("$isodate $user reanimated (login_check)");
+            logv("$user reanimated (login_check)");
           }
         } else {
           rename "$user/@","$user/\@~" unless $opt_d;
@@ -145,7 +147,7 @@ if (chdir $ukeydir and opendir D,'.') {
   while ($file = readdir D) {
     next if $file eq '.' or $file eq '..';
     if (($link = readlink $file and not -e "$link/upload"
-         or -f $file and time > mtime($file)+DS)) {
+         or -f $file and time > lmtime($file)+DS)) {
       logdel($file,".ukeys/$file deleted");
     }
   }
@@ -155,7 +157,7 @@ if (chdir $ukeydir and opendir D,'.') {
 # clean up authorization key lookup directory
 if (chdir $akeydir and opendir D,'.') {
   while ($file = readdir D) {
-    if (-l $file and time > mtime($file)+DS) {
+    if (-l $file and time > (lmtime($file)||0)+DS) {
       logdel($file,".akeys/$file deleted");
     }
   }
@@ -176,7 +178,7 @@ if (chdir $xkeydir and opendir D,'.') {
 # clean up lock directory
 if (chdir $lockdir and opendir D,'.') {
   while ($file = readdir D) {
-    if (-f $file and time > mtime($file)+DS) {
+    if (-f $file and time > lmtime($file)+DS) {
       logdel($file,".locks/$file deleted");
     }
   }
@@ -187,7 +189,7 @@ if (chdir $lockdir and opendir D,'.') {
 if (chdir "$spooldir/.error" and opendir D,'.') {
   while ($file = readdir D) {
     if (-f $file) {
-      $mtime = mtime($file);
+      $mtime = lmtime($file);
       if ($mtime and $today > 10*$keep_default*DS+$mtime) {
         if ($opt_d) { print "unlink .error/$file\n" }
         else        { logdel($file,".error/$file deleted") }
@@ -198,10 +200,10 @@ if (chdir "$spooldir/.error" and opendir D,'.') {
 }
 
 # clean up debug directory
-if (chdir "$logdir/.debug" and opendir D,'.') {
+if (chdir "$spooldir/.debug" and opendir D,'.') {
   while ($file = readdir D) {
     if (-f $file) {
-      $mtime = mtime($file);
+      $mtime = lmtime($file);
       if ($mtime and $today > $keep_default*DS+$mtime) {
         # logdel($file,".debug/$file deleted");
         if ($opt_d) { print "unlink .debug/$file\n" }
@@ -257,7 +259,7 @@ foreach $subuser (glob '*/@MAINUSER') {
 # clean up old OKEYs
 chdir $spooldir;
 foreach my $okey (glob '*/@OKEY/*') {
-  if (time > mtime($okey)+30*DS) {
+  if (time > lmtime($okey)+30*DS) {
     logdel($okey,"$okey deleted");
   }
 }
@@ -301,7 +303,7 @@ if (chdir $gkeydir and opendir D,'.') {
 if (chdir "$spooldir/.reg" and opendir D,'.') {
   while ($file = readdir D) {
     if (-f $file) {
-      $mtime = mtime($file);
+      $mtime = lmtime($file);
       if ($mtime and $today > $mtime+DS) {
         logdel($file,".reg/$file deleted");
       }
@@ -346,7 +348,7 @@ if ($account_expire and $account_expire =~ /^(\d+)/) {
       next if $user =~ /^(fexmaster|fexmail)/ or $user eq $admin;
       next if -l "$user/.login";
 
-      if (time > mtime($user)+$expire*DS) {
+      if (time > lmtime($user)+$expire*DS) {
         # print "$spooldir/$user\n";
         my $locale = readlink "$user/\@LOCALE";
         $locale = 'english' unless $locale and $reactivation{$locale};
@@ -358,8 +360,6 @@ if ($account_expire and $account_expire =~ /^(\d+)/) {
   }
 }
 
-close L;
-
 # vhosts
 exit if $opt_V;
 if (%vhost) {
@@ -374,7 +374,8 @@ if (%vhost) {
   }
 }
 
-if ($notify_newrelease or not defined $notify_newrelease) {
+if ($notify_newrelease and $notify_newrelease !~ /^no$/i 
+    or not defined $notify_newrelease) {
   $notify_newrelease ||= $admin;
   $newnew = $new = '';
   $snew = $FEXHOME.'/doc/new';
@@ -384,10 +385,10 @@ if ($notify_newrelease or not defined $notify_newrelease) {
   else         { $qn = "new?$hostname:0" }
   for (1..3) {
     sleep rand(10);
-    $newnew = `wget -qO- http://fex.rus.uni-stuttgart.de/$qn 2>/dev/null`;
-    last if $newnew =~ /release/;
     $newnew = `wget -qO- http://fex.belwue.de/$qn 2>/dev/null`;
     last if $newnew =~ /release/;
+    # $newnew = `wget -qO- http://fex.rus.uni-stuttgart.de/$qn 2>/dev/null`;
+    # last if $newnew =~ /release/;
   };
   if ($newnew =~ /release/) {
     if ($newnew ne $new) {
@@ -432,12 +433,12 @@ sub cleanup {
   if ($file =~ /\/ADDRESS_BOOK/) {
     logdel($file,"$file deleted");
   } elsif (-d $file and not -f $data) {
-    if ($mtime = mtime("$file/upload")) {
+    if ($mtime = lmtime("$file/upload")) {
       if ($today > $mtime+DS) {
         verbose("rmrf $file (today=$today mtime_upload=$mtime)");
         logdel($file,"$file deleted");
       }
-    } elsif ($mtime = mtime("$file/error")) {
+    } elsif ($mtime = lmtime("$file/error")) {
       if ($today > 3*$keep*DS+$mtime) {
         verbose("rmrf $file (today=$today mtime_error=$mtime keep=$keep)");
         logdel($file,"$file deleted");
@@ -449,19 +450,19 @@ sub cleanup {
     $delay = autodelete($file);
     $delay = 1 if $delay !~ /^\d+$/;
     $delay--;
-    $mtime = mtime($download);
+    $mtime = lmtime($download);
     if ($mtime and $today > $delay*DS+$mtime 
         and logdel($data,"$data deleted")) {
       if (open $ef,'>',$ef) {
         printf {$ef} "%s has been autodeleted after download at %s\n",
-                     filename($file),isodate(mtime($download));
+                     filename($file),isodate(lmtime($download));
         close $ef;
       }
     }
   } elsif (-f $data) {
     my $reactivation = $file =~ m{/\Q$admin/reactivation.txt\E$};
     $warn = $reactivation ? $keep-5 : $keep-2;
-    $mtime = mtime("$file/filename") || mtime($data) || 0;
+    $mtime = lmtime("$file/filename") || lmtime($data) || 0;
     if ($today > $mtime+$keep*DS) {
       if ($account_expire and $reactivation) {
         if ($account_expire =~ /delete/) {
@@ -514,7 +515,7 @@ sub cleanup {
         chomp ($comment = <$c>||'');
         close $c;
       }
-      &{$notify{$locale}}(
+      if (&{$notify{$locale}}(
         status     => 'remind',
         dkey       => $dkey,
         filename   => filename($file),
@@ -522,10 +523,13 @@ sub cleanup {
         comment    => $comment,
         warn       => int(($mtime-$today)/DS)+$keep,
         autodelete => autodelete($file),
-      );
-      open $notify,'>',$notify;
-      close $notify;
-      print "sent reminder for $file\n" if -t or $opt_v;
+      )) {
+        open $notify,'>',$notify;
+        close $notify;
+        print "sent reminder for $file\n" if -t or $opt_v;
+      } else {
+        warn "$0: reminder notification for $file failed\n";
+      }
     }
   }
 }
@@ -545,11 +549,6 @@ sub autodelete {
   return $autodelete||$::autodelete;
 }
 
-sub mtime {
-  my @s = lstat shift;
-  return @s ? $s[9] : undef;
-}
-
 sub logdel {
   my ($file,$msg) = @_;
   my $status = 0;
@@ -560,8 +559,8 @@ sub logdel {
     if ($status = rmrf($file)) {
       logv($msg);
     } else {
-      print L "$isodate $file DEL FAILED : $!\n";
-      warn     "$file DEL FAILED : $!\n" if -t or $opt_v;
+      logv("$file DEL FAILED : $!");
+      warn "$file DEL FAILED : $!\n" if -t or $opt_v;
     }
   }
   
@@ -571,8 +570,17 @@ sub logdel {
 
 sub logv {
   my $msg = shift;
-  print L "$isodate $msg\n" unless $opt_d;
+
   print "$msg\n" if -t or $opt_v;
+
+  unless ($opt_d) {
+    foreach my $ld (@logdir) {
+      if (open my $log,">>$ld/cleanup.log") {
+        print {$log} "$isodate $msg\n";
+        close $log;
+      }
+    }
+  }
 }
 
 
@@ -585,3 +593,9 @@ sub verbose {
     }
   }
 }
+
+
+sub lmtime {
+  my @s = lstat(shift);
+  return @s?$s[9]:0;
+}