X-Git-Url: https://git.adam-barratt.org.uk/?p=mirror%2Fuserdir-ldap-cgi.git;a=blobdiff_plain;f=update.cgi;h=3cf1900f1ac8e1cd01bac60dc08fc6077ee4cf61;hp=406d8134cd1600dd6c160c3af41ed5df9d6674aa;hb=HEAD;hpb=cee6f4d1f3276c419e1485bd840a3345307c9142

diff --git a/update.cgi b/update.cgi
index 406d813..3cf1900 100755
--- a/update.cgi
+++ b/update.cgi
@@ -40,7 +40,7 @@ if ($authtoken || $id) {
 if ($proto eq "http" || !($id && $password)) {
   print "Location: https://$ENV{SERVER_NAME}/$config{webloginhtml}\n\n";
   exit;
-} 
+}
 
 my $ldap;
 
@@ -74,7 +74,7 @@ if (!$auth) {
 $mesg = $ldap->search(base   => $editdn,
                       filter => "uid=*");
 $mesg->code && &Util::HTMLError($mesg->error);
-  
+
 my $entries = $mesg->as_struct;
 if ($mesg->count != 1) {
   # complain and quit
@@ -90,7 +90,7 @@ my $entry = $entries->{$dns[0]};
 if (!($query->param('doupdate'))) {
   # Not yet update, just fill in the form with the current values
   my %data;
-  
+
   # Fill in %data
   # First do the easy stuff - this catches most of the cases
   foreach (keys(%$entry)) {
@@ -98,8 +98,6 @@ if (!($query->param('doupdate'))) {
     $data{$_} = CGI::escapeHTML($data{$_}) if defined $data{$_};
   }
 
-  $data{gender} = 9 if not $data{gender};
-
   # Now we have to fill in the rest that needs some processing...
   $data{id} = $id;
   $data{authtoken} = $authtoken;
@@ -130,18 +128,9 @@ if (!($query->param('doupdate'))) {
   }
 
   $data{mailcontentinspectionaction} ||= 'reject';
-  
+
   $data{email} = CGI::escapeHTML(join(", ", @{$entry->{emailforward}}));
 
-  my $genderselect = '<select name="gender">'
-		   . '<option value="9"'
-		   . ($data{gender} == 9 ? ' selected' : '')
-		   . '>unspecified'
-		   . '<option value="1"'
-		   . ($data{gender} == 1 ? ' selected' : '')
-		   . '>male<option value="2"'
-		   . ($data{gender} == 2 ? ' selected' : '')
-                   . '>female</select>';
   my $mailcontentselect = '<select name="mailcontentinspection">'
                         . '<option value="blackhole"'
                         . ($data{mailcontentinspectionaction} eq 'blackhole' ? ' selected' : '')
@@ -201,7 +190,6 @@ if (!($query->param('doupdate'))) {
   open (F, "<$config{webupdatehtml}") || &Util::HTMLError($!);
   while (<F>) {
     s/~(.+?)~/$data{$1}/g;
-    s/<\?genderselect>/$genderselect/;
     s/<\?mailcontentselect>/$mailcontentselect/;
     s/<\?sudopassword>/$sudopassword/;
     s/<\?sudopasswordhosts>/$sudopasswordhosts/;
@@ -220,17 +208,17 @@ if (!($query->param('doupdate'))) {
 
   # Actually update stuff...
   my ($newpassword, $newstaddress, $newwebpassword, $newrtcpassword);
-  
+
   # 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
   # directly.
   # &Util::FixParams($query);
 
-  if (($query->param('labeleduri')) && 
+  if (($query->param('labeleduri')) &&
       ($query->param('labeleduri') !~ /^https?:\/\//i)) {
     &Util::HTMLError("Your homepage URL is invalid");
   }
-  
+
   if ($query->param('newpass') && $query->param('newpassvrfy')) {
     if ($query->param('newpass') ne $query->param('newpassvrfy')) {
       # passwords don't match...
@@ -244,10 +232,10 @@ if (!($query->param('doupdate'))) {
 
     # create a md5 crypted password
     $newpassword = '{crypt}'.crypt($query->param('newpass'), &Util::CreateCryptSalt(1));
-    
+
     &Util::LDAPUpdate($ldap, $editdn, 'userPassword', $newpassword);
     &Util::UpdateAuthToken($authtoken, $query->param('newpass'));
-  }  
+  }
 
   if ($query->param('newwebpass') && $query->param('newwebpassvrfy')) {
     if ($query->param('newwebpass') ne $query->param('newwebpassvrfy')) {
@@ -262,9 +250,9 @@ if (!($query->param('doupdate'))) {
 
     # create a md5 crypted password
     $newwebpassword = apache_md5_crypt($query->param('newwebpass'), &Util::CreateCryptSalt(1, 1));
-    
+
     &Util::LDAPUpdate($ldap, $editdn, 'webPassword', $newwebpassword);
-  }  
+  }
   if ($query->param('newrtcpass') && $query->param('newrtcpassvrfy')) {
     if ($query->param('newrtcpass') ne $query->param('newrtcpassvrfy')) {
       # passwords don't match...
@@ -279,16 +267,11 @@ if (!($query->param('doupdate'))) {
     $newrtcpassword = &md5_hex( ldap_explode_dn($editdn)->[0]{UID} . '@debian.org:rtc.debian.org:' . $query->param('newrtcpass') );
 
     &Util::LDAPUpdate($ldap, $editdn, 'rtcPassword', $newrtcpassword);
-  }  
+  }
 
   $newstaddress = $query->param('staddress');
   $newstaddress =~ s/\n/\$/m;
 
-  my $gender = $query->param('gender');
-  if ($gender != 1 && $gender != 2) {
-    $gender = 9; # unspecified
-  }
-  
   my ($bd_ok, $bd_yr, $bd_mo, $bd_day);
 
   if ($query->param('birthdate') =~ /^([1-9][0-9]{3})([01][0-9])([0-3][0-9])$/) {
@@ -314,7 +297,7 @@ if (!($query->param('doupdate'))) {
     $bd_ok = 1;
   }
   my ($lat, $long);
-  ($lat, $long) = &Util::CheckLatLong($query->param('latitude'), 
+  ($lat, $long) = &Util::CheckLatLong($query->param('latitude'),
                                       $query->param('longitude'));
   my ($greylisting, $callout, $mailcontentinspection, $defaultoptions);
 
@@ -370,7 +353,7 @@ if (!($query->param('doupdate'))) {
   my %delete_uuids = map { s/^sudopassword-delete-//; $_ => 1} grep { $query->param($_) eq 'delete' } grep { /^sudopassword-delete-/ } $query->param;
   my @keepsudo;
   for my $entry (@{$entry->{'sudopassword'}}) {
-    my ($uuid, $status, $hosts, $crypted) = ($entry =~ /^([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}) (confirmed:[0-9a-f]{40}|unconfirmed) ([a-z0-9.,*]+) ([^ ]+)$/);
+    my ($uuid, $status, $hosts, $crypted) = ($entry =~ /^([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}) (confirmed:[0-9a-f]{40}|unconfirmed) ([a-z0-9.,*-]+) ([^ ]+)$/);
     next unless defined ($uuid);
     next if (defined $delete_uuids{$uuid});
     my %hosts = map { $_ => 1 } split(/,/, $hosts);
@@ -399,7 +382,6 @@ if (!($query->param('doupdate'))) {
   &Util::LDAPUpdate($ldap, $editdn, 'jabberJID', $query->param('jabberjid'));
   &Util::LDAPUpdate($ldap, $editdn, 'labeledURI', $query->param('labeleduri'));
   &Util::LDAPUpdate($ldap, $editdn, 'onVacation', $query->param('onvacation'));
-  &Util::LDAPUpdate($ldap, $editdn, 'gender', $gender);
   &Util::LDAPUpdate($ldap, $editdn, 'birthDate', $query->param('birthdate')) if $bd_ok;
   &Util::LDAPUpdate($ldap, $editdn, 'mailDisableMessage', $query->param('maildisablemessage'));
   &Util::LDAPUpdate($ldap, $editdn, 'mailCallout', $callout);
@@ -410,7 +392,7 @@ if (!($query->param('doupdate'))) {
 
   # when we are done, reload the page with the updated details.
   my $url = "https://$ENV{SERVER_NAME}/$config{webupdateurl}?id=$id;authtoken=$authtoken";
-  print "Location: $url\n\n";  
+  print "Location: $url\n\n";
 }
 
 $ldap->unbind;