[LON-CAPA-cvs] cvs: loncom /auth lonracc.pm

[albertel]

albertel		Tue May 27 14:07:22 2003 EDT

  Modified files:              
    /loncom/auth	lonracc.pm 
  Log:
  - ctually declare variables in proper scope.
  - shows list of all non subscribed hosts if it can't find a subscribed host
  
  
Index: loncom/auth/lonracc.pm
diff -u loncom/auth/lonracc.pm:1.8 loncom/auth/lonracc.pm:1.9
--- loncom/auth/lonracc.pm:1.8	Fri May 23 03:13:27 2003
+++ loncom/auth/lonracc.pm	Tue May 27 14:07:22 2003
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Access Handler for File Transfers
 #
-# $Id: lonracc.pm,v 1.8 2003/05/23 07:13:27 albertel Exp $
+# $Id: lonracc.pm,v 1.9 2003/05/27 18:07:22 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -68,8 +68,9 @@
           $r->log_reason("Could not find host tab file");
           return FORBIDDEN;
        }
+       my $return;
+       my @ids=();
        while ($readline=<$fh>) {
-	  my $return;
           my ($id,$domain,$role,$name,$ip)=split(/:/,$readline);
           if ($name =~ /$reqhost/i) {
               my $filename=$r->filename;
@@ -79,11 +80,12 @@
 	         return OK;
               } else {
                  $return=FORBIDDEN;
+		 push(@ids,$id);
               }
           }
        }
        if ($return == FORBIDDEN) {
-	   $r->log_reason("$id not subscribed", $r->filename);
+	   $r->log_reason(join(':',@ids)." not subscribed", $r->filename);
        }
     }
     $r->log_reason("Invalid request for file transfer from $reqhost",