X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=update.cgi;h=f1f82f9e81ae38e8d62fd6bfa6fdd3246fdbb654;hb=d94bff4b6491c3207152cde4d5fd0aeb67abf0ca;hp=c42eddce278ea1c97a0552464fca34b5cdf00ea7;hpb=3bce39d94939aaf999189e4aaed805858a996153;p=mirror%2Fuserdir-ldap-cgi.git diff --git a/update.cgi b/update.cgi index c42eddc..f1f82f9 100755 --- a/update.cgi +++ b/update.cgi @@ -13,6 +13,7 @@ use Digest::HMAC_SHA1 qw(hmac_sha1_hex); use Util; use English; use URI::Escape; +use Crypt::PasswdMD5; use Net::LDAP qw(LDAP_SUCCESS LDAP_PROTOCOL_ERROR); my %config = &Util::ReadConfigFile; @@ -206,7 +207,7 @@ if (!($query->param('doupdate'))) { # Actually update stuff... - my ($newpassword, $newstaddress); + my ($newpassword, $newstaddress, $newwebpassword); # Good god, why would we want to do that here? it breaks password setting # etc, and it doesn't prevent people from setting eveil stuff in ldap @@ -236,6 +237,23 @@ if (!($query->param('doupdate'))) { &Util::UpdateAuthToken($authtoken, $query->param('newpass')); } + if ($query->param('newwebpass') && $query->param('newwebpassvrfy')) { + if ($query->param('newwebpass') ne $query->param('newwebpassvrfy')) { + # passwords don't match... + &Util::HTMLError("The web-passwords you specified do not match. Please go back and try again."); + } + + my ($r, $msg) = &Util::checkPasswordQuality($query->param('newwebpass'), undef, [@ldapinfo_for_pwcheck]); + if ($r) { + &Util::HTMLError("Password check failed for web-password: $msg. Please go back and try again."); + } + + # create a md5 crypted password + $newwebpassword = apache_md5_crypt($query->param('newwebpass'), &Util::CreateCryptSalt(1, 1)); + + &Util::LDAPUpdate($ldap, $editdn, 'webPassword', $newwebpassword); + } + $newstaddress = $query->param('staddress'); $newstaddress =~ s/\n/\$/m;