phpLDAPadmin misconfiguration

Bug #333620 reported by TDJACR on 2009-02-24
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
phpLDAPadmin
New
Low
phpldapadmin (Ubuntu)
Low
Unassigned

Bug Description

Binary package hint: phpldapadmin

While browsing the default phpldapadmin configuration, I found what seems to be an error.

/* The port your LDAP server listens on (no quotes). 389 is standard. */
$ldapservers->SetValue($i,'server','port','389');

It clearly says no quotes, but on installation, the line contains single quotes around the port number.
Maybe I am missing something, but this looks like a minor bug to me.

Andreas Olsson (andol) wrote :

Actually, this might very well not be an error in the configuration, but in the comment.

http://phpldapadmin.sourceforge.net/wiki/index.php/LDAP_server_definitions

Still, there is a bug here, we just have to figure out which one :) No matter what, there is an inconsistency.

I downloaded the most recent version of directly from phpldapadmin (upstream) and they have exactly the same writing.

Changed in phpldapadmin:
status: New → Incomplete
Andreas Olsson (andol) wrote :
Changed in phpldapadmin:
status: Unknown → New
Andreas Moog (ampelbein) wrote :

Fixed upstream, thanks for pointing to it.

Changed in phpldapadmin (Ubuntu):
importance: Undecided → Low
status: Incomplete → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package phpldapadmin - 1.1.0.7-1ubuntu1

---------------
phpldapadmin (1.1.0.7-1ubuntu1) karmic; urgency=low

  [ Adam Sommer ]
  * Merge from debian unstable, remaining changes (LP: #376766):
    - Merged call to dh_install to install debian/additional-templates/*
    - Adds php_value memory_limit 32M to the apache.conf.
  * Configuration error fix. (LP: #333620)
  * Alias objects now work. (LP: #223842)

phpldapadmin (1.1.0.7-1) unstable; urgency=low

  * New upstream release.
  * debian/po/es.po: added. (Closes: #520504)

phpldapadmin (1.1.0.6-1) unstable; urgency=low

  * New upstream release. (Closes: #518578)
  * debian/rules: removed "-m 644" from the dh_install call. (Closes: #518847)
  * debian/postrm: remove config.php at purge time. (Closes: #519086)
  * debian/patches/hungarian.dpatch: fixed a hungarian translation.
    (Closes: #505559)

 -- Adam Sommer <email address hidden> Thu, 14 May 2009 22:05:53 -0400

Changed in phpldapadmin (Ubuntu):
status: Fix Committed → Fix Released
Changed in phpldapadmin:
importance: Unknown → Low
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.