From: Martin Zobel-Helas <zobel@debian.org>
Date: Thu, 8 Mar 2012 18:23:11 +0000 (+0100)
Subject: add webpassword
X-Git-Tag: release-0.3.35~2
X-Git-Url: https://git.adam-barratt.org.uk/?p=mirror%2Fuserdir-ldap-cgi.git;a=commitdiff_plain;h=a5c0027c487d7b6e1ed6a82dbe3bbdebbaa51c12

add webpassword
Signed-off-by: Martin Zobel-Helas <zobel@debian.org>
---

diff --git a/debian/changelog b/debian/changelog
index 87f9ffd..c79ca00 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,8 +1,8 @@
 userdir-ldap-cgi (0.3.35) Xnstable; urgency=low
 
-  * 
+  * add webPassword handling
 
- -- Martin Zobel-Helas <zobel@debian.org>  Fri, 06 Jan 2012 13:01:30 +0100
+ -- Martin Zobel-Helas <zobel@debian.org>  Thu, 08 Mar 2012 19:24:05 +0100
 
 userdir-ldap-cgi (0.3.34) unstable; urgency=low
 
diff --git a/html/update.wml b/html/update.wml
index 05ebf74..184004f 100644
--- a/html/update.wml
+++ b/html/update.wml
@@ -189,6 +189,13 @@
     <input name=maildisablemessage value="~maildisablemessage~" size=30>
   </td></tr>
 
+  <tr><td class=right>
+    <b>Change web password:</b><br><font size="-1">(re-enter to verify)</font>
+  </td><td>
+    <input size=30 name=newwebpass type=password><br>
+    <input size=30 name=newwebpassvrfy type=password>
+  </td></tr>
+
   <tr><td class="left" colspan=2>
     <b>sudo passwords:</b><br>
     <table border=2>
diff --git a/update.cgi b/update.cgi
index c42eddc..f0ff1ac 100755
--- a/update.cgi
+++ b/update.cgi
@@ -206,7 +206,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 +236,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 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: $msg.  Please go back and try again.");
+    }
+
+    # create a md5 crypted password
+    $newwebpassword = '{crypt}'.crypt($query->param('newwebpass'), &Util::CreateCryptSalt(1));
+    
+    &Util::LDAPUpdate($ldap, $editdn, 'webPassword', $newwebpassword);
+  }  
+
   $newstaddress = $query->param('staddress');
   $newstaddress =~ s/\n/\$/m;