--- loncom/lond	2016/08/11 09:52:39	1.489.2.20
+++ loncom/lond	2016/07/25 19:49:45	1.523
@@ -2,7 +2,7 @@
 # The LearningOnline Network
 # lond "LON Daemon" Server (port "LOND" 5663)
 #
-# $Id: lond,v 1.489.2.20 2016/08/11 09:52:39 raeburn Exp $
+# $Id: lond,v 1.523 2016/07/25 19:49:45 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -41,7 +41,7 @@ use IO::File;
 #use Apache::File;
 use POSIX;
 use Crypt::IDEA;
-use LWP::UserAgent();
+use HTTP::Request;
 use Digest::MD5 qw(md5_hex);
 use GDBM_File;
 use Authen::Krb5;
@@ -55,13 +55,17 @@ use LONCAPA::lonssl;
 use Fcntl qw(:flock);
 use Apache::lonnet;
 use Mail::Send;
+use Crypt::Eksblowfish::Bcrypt;
+use Digest::SHA;
+use Encode;
+use LONCAPA::LWPReq;
 
 my $DEBUG = 0;		       # Non zero to enable debug log entries.
 
 my $status='';
 my $lastlog='';
 
-my $VERSION='$Revision: 1.489.2.20 $'; #' stupid emacs
+my $VERSION='$Revision: 1.523 $'; #' stupid emacs
 my $remoteVERSION;
 my $currenthostid="default";
 my $currentdomainid;
@@ -664,10 +668,8 @@ sub PushFile {
                 $clientprotocol = 'http' if ($clientprotocol ne 'https');
                 my $url = '/adm/'.$filename;
                 $url =~ s{_}{/};
-                my $ua=new LWP::UserAgent;
-                $ua->timeout(60);
                 my $request=new HTTP::Request('GET',"$clientprotocol://$clienthost$url");
-                my $response=$ua->request($request);
+                my $response = LONCAPA::LWPReq::makerequest($clientname,$request,'',\%perlvar,60,0);
                 if ($response->is_error()) {
                     &logthis('<font color="red"> Pushfile: unable to install '
                             .$tablefile." - error attempting to pull data. </font>");
@@ -1794,6 +1796,16 @@ sub server_distarch_handler {
 }
 &register_handler("serverdistarch", \&server_distarch_handler, 0, 1, 0);
 
+sub server_certs_handler {
+    my ($cmd,$tail,$client) = @_;
+    my $userinput = "$cmd:$tail";
+    my $result;
+    my $result = &LONCAPA::Lond::server_certs(\%perlvar);
+    &Reply($client,\$result,$userinput);
+    return;
+}
+&register_handler("servercerts", \&server_certs_handler, 0, 1, 0);
+
 #   Process a reinit request.  Reinit requests that either
 #   lonc or lond be reinitialized so that an updated 
 #   host.tab or domain.tab can be processed.
@@ -1932,12 +1944,8 @@ sub authenticate_handler {
                 if (ref($hostedsession) eq 'HASH') {
                     $hosted = $hostedsession->{'hosted'};
                 }
-                my $loncaparev = $clientversion;
-                if ($loncaparev eq '') {
-                    $loncaparev = $Apache::lonnet::loncaparevs{$clientname};
-                }
                 $canhost = &Apache::lonnet::can_host_session($udom,$clientname,
-                                                             $loncaparev,
+                                                             $clientversion,
                                                              $remote,$hosted);
             }
         }
@@ -2017,15 +2025,14 @@ sub change_password_handler {
 	my ($howpwd,$contentpwd)=split(/:/,$realpasswd);
 	if ($howpwd eq 'internal') {
 	    &Debug("internal auth");
-	    my $salt=time;
-	    $salt=substr($salt,6,2);
-	    my $ncpass=crypt($npass,$salt);
+            my $ncpass = &hash_passwd($udom,$npass);
 	    if(&rewrite_password_file($udom, $uname, "internal:$ncpass")) {
 		my $msg="Result of password change for $uname: pwchange_success";
                 if ($lonhost) {
                     $msg .= " - request originated from: $lonhost";
                 }
                 &logthis($msg);
+                &update_passwd_history($uname,$udom,$howpwd,$context);
 		&Reply($client, "ok\n", $userinput);
 	    } else {
 		&logthis("Unable to open $uname passwd "               
@@ -2034,6 +2041,9 @@ sub change_password_handler {
 	    }
 	} elsif ($howpwd eq 'unix' && $context ne 'reset_by_email') {
 	    my $result = &change_unix_password($uname, $npass);
+            if ($result eq 'ok') {
+                &update_passwd_history($uname,$udom,$howpwd,$context);
+             }
 	    &logthis("Result of password change for $uname: ".
 		     $result);
 	    &Reply($client, \$result, $userinput);
@@ -2056,6 +2066,42 @@ sub change_password_handler {
 }
 &register_handler("passwd", \&change_password_handler, 1, 1, 0);
 
+sub hash_passwd {
+    my ($domain,$plainpass,@rest) = @_;
+    my ($salt,$cost);
+    if (@rest) {
+        $cost = $rest[0];
+        # salt is first 22 characters, base-64 encoded by bcrypt
+        my $plainsalt = substr($rest[1],0,22);
+        $salt = Crypt::Eksblowfish::Bcrypt::de_base64($plainsalt);
+    } else {
+        my $defaultcost;
+        my %domconfig =
+            &Apache::lonnet::get_dom('configuration',['password'],$domain);
+        if (ref($domconfig{'password'}) eq 'HASH') {
+            $defaultcost = $domconfig{'password'}{'cost'};
+        }
+        if (($defaultcost eq '') || ($defaultcost =~ /D/)) {
+            $cost = 10;
+        } else {
+            $cost = $defaultcost;
+        }
+        # Generate random 16-octet base64 salt
+        $salt = "";
+        $salt .= pack("C", int rand(256)) for 1..16;
+    }
+    my $hash = &Crypt::Eksblowfish::Bcrypt::bcrypt_hash({
+        key_nul => 1,
+        cost    => $cost,
+        salt    => $salt,
+    }, Digest::SHA::sha512(Encode::encode('UTF-8',$plainpass)));
+
+    my $result = join("!", "", "bcrypt", sprintf("%02d",$cost),
+                &Crypt::Eksblowfish::Bcrypt::en_base64($salt).
+                &Crypt::Eksblowfish::Bcrypt::en_base64($hash));
+    return $result;
+}
+
 #
 #   Create a new user.  User in this case means a lon-capa user.
 #   The user must either already exist in some authentication realm
@@ -2099,7 +2145,8 @@ sub add_user_handler {
 		    ."makeuser";
 	    }
 	    unless ($fperror) {
-		my $result=&make_passwd_file($uname,$udom,$umode,$npass, $passfilename);
+		my $result=&make_passwd_file($uname,$udom,$umode,$npass,
+                                             $passfilename,'makeuser');
 		&Reply($client,\$result, $userinput);     #BUGBUG - could be fail
 	    } else {
 		&Failure($client, \$fperror, $userinput);
@@ -2168,12 +2215,14 @@ sub change_authentication_handler {
 		my $result = &change_unix_password($uname, $npass);
 		&logthis("Result of password change for $uname: ".$result);
 		if ($result eq "ok") {
+                    &update_passwd_history($uname,$udom,$umode,'changeuserauth'); 
 		    &Reply($client, \$result);
 		} else {
 		    &Failure($client, \$result);
 		}
 	    } else {
-		my $result=&make_passwd_file($uname,$udom,$umode,$npass,$passfilename);
+		my $result=&make_passwd_file($uname,$udom,$umode,$npass,
+                                             $passfilename,'changeuserauth');
 		#
 		#  If the current auth mode is internal, and the old auth mode was
 		#  unix, or krb*,  and the user is an author for this domain,
@@ -2194,6 +2243,17 @@ sub change_authentication_handler {
 }
 &register_handler("changeuserauth", \&change_authentication_handler, 1,1, 0);
 
+sub update_passwd_history {
+    my ($uname,$udom,$umode,$context) = @_;
+    my $proname=&propath($udom,$uname);
+    my $now = time;
+    if (open(my $fh,">>$proname/passwd.log")) {
+        print $fh "$now:$umode:$context\n";
+        close($fh);
+    }
+    return;
+}
+
 #
 #   Determines if this is the home server for a user.  The home server
 #   for a user will have his/her lon-capa passwd file.  Therefore all we need
@@ -2280,9 +2340,8 @@ sub update_resource_handler {
 # FIXME: this should use the LWP mechanism, not internal alarms.
                 alarm(1200);
 		{
-		    my $ua=new LWP::UserAgent;
 		    my $request=new HTTP::Request('GET',"$remoteurl");
-		    $response=$ua->request($request,$transname);
+                    $response=&LONCAPA::LWPReq::makerequest($clientname,$request,$transname,\%perlvar,1200,0,1);
 		}
 		alarm(0);
 		if ($response->is_error()) {
@@ -2295,9 +2354,8 @@ sub update_resource_handler {
 # FIXME: isn't there an internal LWP mechanism for this?
 			alarm(120);
 			{
-			    my $ua=new LWP::UserAgent;
 			    my $mrequest=new HTTP::Request('GET',$remoteurl.'.meta');
-			    my $mresponse=$ua->request($mrequest,$fname.'.meta');
+                            my $mresponse = &LONCAPA::LWPReq::makerequest($clientname,$mrequest,$fname.'.meta',\%perlvar,120,0,1);
 			    if ($mresponse->is_error()) {
 				unlink($fname.'.meta');
 			    }
@@ -2372,11 +2430,15 @@ sub fetch_user_file_handler {
 	my $remoteurl=$clientprotocol.'://'.$clienthost.'/userfiles/'.$fname;
 	my $response;
 	Debug("Remote URL : $remoteurl Transfername $transname Destname: $destname");
-	alarm(120);
+	alarm(1200);
 	{
-	    my $ua=new LWP::UserAgent;
 	    my $request=new HTTP::Request('GET',"$remoteurl");
-	    $response=$ua->request($request,$transname);
+            my $verifycert = 1;
+            my @machine_ids = &Apache::lonnet::current_machine_ids();
+            if (grep(/^\Q$clientname\E$/,@machine_ids)) {
+                $verifycert = 0;
+            }
+            $response = &LONCAPA::LWPReq::makerequest($clientname,$request,$transname,\%perlvar,1200,$verifycert);
 	}
 	alarm(0);
 	if ($response->is_error()) {
@@ -2445,13 +2507,13 @@ sub remove_user_file_handler {
 	    if (-e $file) {
 		#
 		#   If the file is a regular file unlink is fine...
-		#   However it's possible the client wants a dir
-		#   removed, in which case rmdir is more appropriate
-	        #   Note: rmdir will only remove an empty directory.
+		#   However it's possible the client wants a dir 
+		#   removed, in which case rmdir is more appropriate.
+		#   Note: rmdir will only remove an empty directory.
 		#
 	        if (-f $file){
 		    unlink($file);
-                    # for html files remove the associated .bak file
+                    # for html files remove the associated .bak file 
                     # which may have been created by the editor.
                     if ($ufile =~ m{^((docs|supplemental)/(?:\d+|default)/\d+(?:|/.+)/)[^/]+\.x?html?$}i) {
                         my $path = $1;
@@ -2825,8 +2887,8 @@ sub newput_user_profile_entry {
                 &logthis("error: ".($!+0)." untie (GDBM) failed ".
                          "while attempting newput - early out as key exists");
             }
-	    &Failure($client, "key_exists: ".$key."\n",$userinput);
-	    return 1;
+            &Failure($client, "key_exists: ".$key."\n",$userinput);
+            return 1;
 	}
     }
 
@@ -3228,6 +3290,17 @@ sub get_profile_keys {
 sub dump_profile_database {
     my ($cmd, $tail, $client) = @_;
 
+    my $res = LONCAPA::Lond::dump_profile_database($tail);
+
+    if ($res =~ /^error:/) {
+        Failure($client, \$res, "$cmd:$tail");
+    } else {
+        Reply($client, \$res, "$cmd:$tail");
+    }
+
+    return 1;  
+
+    #TODO remove 
     my $userinput = "$cmd:$tail";
    
     my ($udom,$uname,$namespace) = split(/:/,$tail);
@@ -3307,11 +3380,11 @@ sub dump_with_regexp {
     my ($cmd, $tail, $client) = @_;
 
     my $res = LONCAPA::Lond::dump_with_regexp($tail, $clientversion);
-
+    
     if ($res =~ /^error:/) {
-        &Failure($client, \$res, "$cmd:$tail");
+        Failure($client, \$res, "$cmd:$tail");
     } else {
-        &Reply($client, \$res, "$cmd:$tail");
+        Reply($client, \$res, "$cmd:$tail");
     }
 
     return 1;
@@ -3349,7 +3422,6 @@ sub store_handler {
     my ($cmd, $tail, $client) = @_;
  
     my $userinput = "$cmd:$tail";
-
     chomp($tail);
     my ($udom,$uname,$namespace,$rid,$what,$laststore) =split(/:/,$tail);
     if ($namespace ne 'roles') {
@@ -3379,7 +3451,6 @@ sub store_handler {
                     $numtrans =~ s/D//g;
                 }
             }
-
 	    $hashref->{"version:$rid"}++;
 	    my $version=$hashref->{"version:$rid"};
 	    my $allkeys=''; 
@@ -3396,7 +3467,7 @@ sub store_handler {
                 if ($numtrans) {
                     $msg = 'delay:'.$numtrans;
                 }
-                &Reply($client, "$msg\n", $userinput);
+		&Reply($client, "$msg\n", $userinput);
 	    } else {
 		&Failure($client, "error: ".($!+0)." untie(GDBM) Failed ".
 			"while attempting store\n", $userinput);
@@ -3913,7 +3984,7 @@ sub put_course_id_hash_handler {
 #
 #                 domcloner - flag to indicate if user can create CCs in course's domain.
 #                             If so, ability to clone course is automatic.
-#                 hasuniquecode - filter by courses for which a six character unique code has
+#                 hasuniquecode - filter by courses for which a six character unique code has 
 #                                 been set.
 #
 #     $client  - The socket open on the client.
@@ -3923,6 +3994,17 @@ sub put_course_id_hash_handler {
 #   a reply is written to $client.
 sub dump_course_id_handler {
     my ($cmd, $tail, $client) = @_;
+
+    my $res = LONCAPA::Lond::dump_course_id_handler($tail);
+    if ($res =~ /^error:/) {
+        Failure($client, \$res, "$cmd:$tail");
+    } else {
+        Reply($client, \$res, "$cmd:$tail");
+    }
+
+    return 1;  
+
+    #TODO remove
     my $userinput = "$cmd:$tail";
 
     my ($udom,$since,$description,$instcodefilter,$ownerfilter,$coursefilter,
@@ -4370,6 +4452,122 @@ sub put_domain_handler {
 }
 &register_handler("putdom", \&put_domain_handler, 0, 1, 0);
 
+# Updates one or more entries in clickers.db file at the domain level
+#
+# Parameters:
+#    $cmd      - The command that got us here.
+#    $tail     - Tail of the command (remaining parameters).
+#                In this case a colon separated list containing:
+#                (a) the domain for which we are updating the entries,
+#                (b) the action required -- add or del -- and
+#                (c) a &-separated list of entries to add or delete.
+#    $client   - File descriptor connected to client.
+# Returns
+#     1        - Continue processing.
+#     0        - Requested to exit, caller should shut down.
+#  Side effects:
+#     reply is written to $client.
+#
+
+
+sub update_clickers {
+    my ($cmd, $tail, $client)  = @_;
+
+    my $userinput = "$cmd:$tail";
+    my ($udom,$action,$what) =split(/:/,$tail,3);
+    chomp($what);
+
+    my $hashref = &tie_domain_hash($udom, "clickers", &GDBM_WRCREAT(),
+                                 "U","$action:$what");
+
+    if (!$hashref) {
+        &Failure( $client, "error: ".($!+0)." tie(GDBM) Failed ".
+                  "while attempting updateclickers\n", $userinput);
+        return 1;
+    }
+
+    my @pairs=split(/\&/,$what);
+    foreach my $pair (@pairs) {
+        my ($key,$value)=split(/=/,$pair);
+        if ($action eq 'add') {
+            if (exists($hashref->{$key})) {
+                my @newvals = split(/,/,&unescape($value));
+                my @currvals = split(/,/,&unescape($hashref->{$key}));
+                my @merged = sort(keys(%{{map { $_ => 1 } (@newvals,@currvals)}}));
+                $hashref->{$key}=&escape(join(',',@merged));
+            } else {
+                $hashref->{$key}=$value;
+            }
+        } elsif ($action eq 'del') {
+            if (exists($hashref->{$key})) {
+                my %current;
+                map { $current{$_} = 1; } split(/,/,&unescape($hashref->{$key}));
+                map { delete($current{$_}); } split(/,/,&unescape($value));
+                if (keys(%current)) {
+                    $hashref->{$key}=&escape(join(',',sort(keys(%current))));
+                } else {
+                    delete($hashref->{$key});
+                }
+            }
+        }
+    }
+    if (&untie_user_hash($hashref)) {
+        &Reply( $client, "ok\n", $userinput);
+    } else {
+        &Failure($client, "error: ".($!+0)." untie(GDBM) failed ".
+                 "while attempting put\n",
+                 $userinput);
+    }
+    return 1;
+}
+&register_handler("updateclickers", \&update_clickers, 0, 1, 0);
+
+
+# Deletes one or more entries in a namespace db file at the domain level
+#
+# Parameters:
+#    $cmd      - The command that got us here.
+#    $tail     - Tail of the command (remaining parameters).
+#                In this case a colon separated list containing:
+#                (a) the domain for which we are deleting the entries,
+#                (b) &-separated list of keys to delete.  
+#    $client   - File descriptor connected to client.
+# Returns
+#     1        - Continue processing.
+#     0        - Requested to exit, caller should shut down.
+#  Side effects:
+#     reply is written to $client.
+#
+
+sub del_domain_handler {
+    my ($cmd,$tail,$client) = @_;
+
+    my $userinput = "$cmd:$tail";
+
+    my ($udom,$namespace,$what)=split(/:/,$tail,3);
+    chomp($what);
+    my $hashref = &tie_domain_hash($udom,$namespace,&GDBM_WRCREAT(),
+                                   "D", $what);
+    if ($hashref) {
+        my @keys=split(/\&/,$what);
+        foreach my $key (@keys) {
+            delete($hashref->{$key});
+        }
+        if (&untie_user_hash($hashref)) {
+            &Reply($client, "ok\n", $userinput);
+        } else {
+            &Failure($client, "error: ".($!+0)." untie(GDBM) Failed ".
+                    "while attempting deldom\n", $userinput);
+        }
+    } else {
+        &Failure( $client, "error: ".($!+0)." tie(GDBM) Failed ".
+                 "while attempting deldom\n", $userinput);
+    }
+    return 1;
+}
+&register_handler("deldom", \&del_domain_handler, 0, 1, 0);
+
+
 # Unencrypted get from the namespace database file at the domain level.
 # This function retrieves a keyed item from a specific named database in the
 # domain directory.
@@ -4529,7 +4727,7 @@ sub get_id_handler {
 # Returns:
 #     1   - Continue processing
 #     0   - Exit server.
-#
+#     
 #
 
 sub del_id_handler {
@@ -5275,6 +5473,51 @@ sub create_auto_enroll_password_handler
 &register_handler("autocreatepassword", \&create_auto_enroll_password_handler, 
 		  0, 1, 0);
 
+sub auto_export_grades_handler {
+    my ($cmd, $tail, $client) = @_;
+    my $userinput = "$cmd:$tail";
+    my ($cdom,$cnum,$info,$data) = split(/:/,$tail);
+    my $inforef = &Apache::lonnet::thaw_unescape($info);
+    my $dataref = &Apache::lonnet::thaw_unescape($data);
+    my ($outcome,$result);;
+    eval {
+        local($SIG{__DIE__})='DEFAULT';
+        my %rtnhash;
+        $outcome=&localenroll::export_grades($cdom,$cnum,$inforef,$dataref,\%rtnhash);
+        if ($outcome eq 'ok') {
+            foreach my $key (keys(%rtnhash)) {
+                $result .= &escape($key).'='.&Apache::lonnet::freeze_escape($rtnhash{$key}).'&';
+            }
+            $result =~ s/\&$//;
+        }
+    };
+    if (!$@) {
+        if ($outcome eq 'ok') {
+            if ($cipher) {
+                my $cmdlength=length($result);
+                $result.="         ";
+                my $encresult='';
+                for (my $encidx=0;$encidx<=$cmdlength;$encidx+=8) {
+                    $encresult.= unpack("H16",
+                                        $cipher->encrypt(substr($result,
+                                                                $encidx,
+                                                                8)));
+                }
+                &Reply( $client, "enc:$cmdlength:$encresult\n", $userinput);
+            } else {
+                &Failure( $client, "error:no_key\n", $userinput);
+            }
+        } else {
+            &Reply($client, "$outcome\n", $userinput);
+        }
+    } else {
+        &Failure($client,"export_error\n",$userinput);
+    }
+    return 1;
+}
+&register_handler("autoexportgrades", \&auto_export_grades_handler,
+                  0, 1, 0);
+
 #   Retrieve and remove temporary files created by/during autoenrollment.
 #
 # Formal Parameters:
@@ -5295,6 +5538,7 @@ sub retrieve_auto_file_handler {
     my ($filename)   = split(/:/, $tail);
 
     my $source = $perlvar{'lonDaemons'}.'/tmp/'.$filename;
+
     if ($filename =~m{/\.\./}) {
         &Failure($client, "refused\n", $userinput);
     } elsif ( (-e $source) && ($filename ne '') ) {
@@ -5329,7 +5573,7 @@ sub crsreq_checks_handler {
     my $userinput = "$cmd:$tail";
     my $dom = $tail;
     my $result;
-    my @reqtypes = ('official','unofficial','community','textbook');
+    my @reqtypes = ('official','unofficial','community','textbook','placement');
     eval {
         local($SIG{__DIE__})='DEFAULT';
         my %validations;
@@ -6374,6 +6618,9 @@ sub Debug {
 #     reply   - Text to send to client.
 #     request - Original request from client.
 #
+#NOTE $reply must be terminated by exactly *one* \n. If $reply is a reference
+#this is done automatically ($$reply must not contain any \n in this case). 
+#If $reply is a string the caller has to ensure this.
 sub Reply {
     my ($fd, $reply, $request) = @_;
     if (ref($reply)) {
@@ -6627,7 +6874,7 @@ sub make_new_child {
             }
         } elsif ($dist =~ /^suse(\d+\.\d+)$/) {
             if (($1 eq '9.3') || ($1 >= 12.2)) {
-                $no_ets = 1;
+                $no_ets = 1; 
             }
         } elsif ($dist =~ /^sles(\d+)$/) {
             if ($1 > 11) {
@@ -6639,8 +6886,8 @@ sub make_new_child {
             }
         }
         unless ($no_ets) {
-            &Authen::Krb5::init_ets();
-        }
+	    &Authen::Krb5::init_ets();
+	}
 
 	&status('Accepted connection');
 # =============================================================================
@@ -6684,12 +6931,12 @@ sub make_new_child {
 		#  If the remote is attempting a local init... give that a try:
 		#
 		(my $i, my $inittype, $clientversion) = split(/:/, $remotereq);
-                # For LON-CAPA 2.9, the  client session will have sent its LON-CAPA
-                # version when initiating the connection. For LON-CAPA 2.8 and older,
-                # the version is retrieved from the global %loncaparevs in lonnet.pm.
-                # $clientversion contains path to keyfile if $inittype eq 'local'
-                # it's overridden below in this case
-                $clientversion ||= $Apache::lonnet::loncaparevs{$clientname};
+        # For LON-CAPA 2.9, the  client session will have sent its LON-CAPA
+        # version when initiating the connection. For LON-CAPA 2.8 and older,
+        # the version is retrieved from the global %loncaparevs in lonnet.pm.            
+        # $clientversion contains path to keyfile if $inittype eq 'local'
+        # it's overridden below in this case
+        $clientversion ||= $Apache::lonnet::loncaparevs{$clientname};
 
 		# If the connection type is ssl, but I didn't get my
 		# certificate files yet, then I'll drop  back to 
@@ -7023,7 +7270,18 @@ sub validate_user {
     } 
     if ($howpwd ne 'nouser') {
 	if($howpwd eq "internal") { # Encrypted is in local password file.
-	    $validated = (crypt($password, $contentpwd) eq $contentpwd);
+            if (length($contentpwd) == 13) {
+                $validated = (crypt($password,$contentpwd) eq $contentpwd);
+                if ($validated) {
+                    my $ncpass = &hash_passwd($domain,$password);
+                    if (&rewrite_password_file($domain,$user,"$howpwd:$ncpass")) {
+                        &update_passwd_history($user,$domain,$howpwd,'conversion');
+                        &logthis("Validated password hashed with bcrypt for $user:$domain");
+                    }
+                }
+            } else {
+                $validated = &check_internal_passwd($password,$contentpwd,$domain);
+            }
 	}
 	elsif ($howpwd eq "unix") { # User is a normal unix user.
 	    $contentpwd = (getpwnam($user))[1];
@@ -7091,6 +7349,39 @@ sub validate_user {
     return $validated;
 }
 
+sub check_internal_passwd {
+    my ($plainpass,$stored,$domain) = @_;
+    my (undef,$method,@rest) = split(/!/,$stored);
+    if ($method eq "bcrypt") {
+        my $result = &hash_passwd($domain,$plainpass,@rest);
+        if ($result ne $stored) {
+            return 0;
+        }
+        # Upgrade to a larger number of rounds if necessary
+        my $defaultcost;
+        my %domconfig =
+            &Apache::lonnet::get_dom('configuration',['password'],$domain);
+        if (ref($domconfig{'password'}) eq 'HASH') {
+            $defaultcost = $domconfig{'password'}{'cost'};
+        }
+        if (($defaultcost eq '') || ($defaultcost =~ /D/)) {
+            $defaultcost = 10;
+        }
+        return 1 unless($rest[0]<$defaultcost);
+    }
+    return 0;
+}
+
+sub get_last_authchg {
+    my ($domain,$user) = @_;
+    my $lastmod;
+    my $logname = &propath($domain,$user).'/passwd.log';
+    if (-e "$logname") {
+        $lastmod = (stat("$logname"))[9];
+    }
+    return $lastmod;
+}
+
 sub krb4_authen {
     my ($password,$null,$user,$contentpwd) = @_;
     my $validated = 0;
@@ -7406,26 +7697,26 @@ sub change_unix_password {
 
 
 sub make_passwd_file {
-    my ($uname,$udom,$umode,$npass,$passfilename)=@_;
+    my ($uname,$udom,$umode,$npass,$passfilename,$action)=@_;
     my $result="ok";
     if ($umode eq 'krb4' or $umode eq 'krb5') {
 	{
 	    my $pf = IO::File->new(">$passfilename");
 	    if ($pf) {
 		print $pf "$umode:$npass\n";
+                &update_passwd_history($uname,$udom,$umode,$action);
 	    } else {
 		$result = "pass_file_failed_error";
 	    }
 	}
     } elsif ($umode eq 'internal') {
-	my $salt=time;
-	$salt=substr($salt,6,2);
-	my $ncpass=crypt($npass,$salt);
+        my $ncpass = &hash_passwd($udom,$npass);
 	{
 	    &Debug("Creating internal auth");
 	    my $pf = IO::File->new(">$passfilename");
 	    if($pf) {
-		print $pf "internal:$ncpass\n"; 
+		print $pf "internal:$ncpass\n";
+                &update_passwd_history($uname,$udom,$umode,$action); 
 	    } else {
 		$result = "pass_file_failed_error";
 	    }
@@ -7506,6 +7797,8 @@ sub get_usersession_config {
 }
 
 
+
+
 sub distro_and_arch {
     return $dist.':'.$arch;
 }
@@ -7689,7 +7982,7 @@ Allow for a password to be set.
 
 Make a user.
 
-=item passwd
+=item changeuserauth
 
 Allow for authentication mechanism and password to be changed.
 
@@ -7778,6 +8071,10 @@ for each student, defined perhaps by the
 Returns usernames corresponding to IDs.  (These "IDs" are unique identifiers
 for each student, defined perhaps by the institutional Registrar.)
 
+=item iddel
+
+Deletes one or more ids in a domain's id database.
+
 =item tmpput
 
 Accept and store information in temporary space.
@@ -7834,6 +8131,8 @@ Authen::Krb5
 
 =head1 COREQUISITES
 
+none
+
 =head1 OSNAMES
 
 linux
@@ -7921,9 +8220,9 @@ or the CA's certificate in the call to l
 <error> is the textual reason this failed.  Usual reasons:
 
 =over 2
-       
+
 =item Apache config file for loncapa  incorrect:
- 
+
 one of the variables 
 lonCertificateDirectory, lonnetCertificateAuthority, or lonnetCertificate
 undefined or incorrect
@@ -8042,7 +8341,7 @@ Could not rewrite the
 internal password file for a user
 
 =item Result of password change for <user> : <result>
-                                                                     
+
 A unix password change for <user> was attempted 
 and the pipe returned <result>  
 
@@ -8071,7 +8370,7 @@ lond has been asked to exit by its clien
 client systemand <input> is the full exit command sent to the server.
 
 =item Red CRITICAL: ABNORMAL EXIT. child <pid> for server <hostname> died through a crass with this error->[<message>].
-                                                 
+
 A lond child terminated.  NOte that this termination can also occur when the
 child receives the QUIT or DIE signals.  <pid> is the process id of the child,
 <hostname> the host lond is working for, and <message> the reason the child died
@@ -8155,7 +8454,7 @@ file when sent it's USR1 signal.  That p
 assumed to be hung in some un-fixable way.
 
 =item Finished checking children                   
- 
+
 Master processs's USR1 processing is cojmplete.
 
 =item (Red) CRITICAL: ------- Starting ------            
@@ -8169,7 +8468,7 @@ Started a new child process for <client>
 connected to the child.  This was as a result of a TCP/IP connection from a client.
 
 =item Unable to determine who caller was, getpeername returned nothing
-                                                  
+
 In child process initialization.  either getpeername returned undef or
 a zero sized object was returned.  Processing continues, but in my opinion,
 this should be cause for the child to exit.
@@ -8180,7 +8479,7 @@ In child process initialization.  The pe
 The client address is stored as "Unavailable" and processing continues.
 
 =item (Yellow) INFO: Connection <ip> <name> connection type = <type>
-                                                  
+
 In child initialization.  A good connectionw as received from <ip>.
 
 =over 2
@@ -8230,7 +8529,7 @@ The client (<client> is the peer's name
 negotiated an SSL connection with this child process.
 
 =item (Green) Successful insecure authentication with <client>
-                                                   
+
 
 The client has successfully negotiated an  insecure connection withthe child process.