<?xml version="1.0" encoding="UTF-8"?> 
<?xml-stylesheet href="https://dev.horde.org/themes/horde//default/feed-rss.xsl" type="text/xsl"?> 
<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom"> 
 <channel> 
  <title>doesn&#039;t properly quote</title> 
  <pubDate>Fri, 10 Apr 2026 20:05:41 +0000</pubDate> 
  <link>https://bugs.horde.org/ticket/1146</link> 
  <atom:link rel="self" type="application/rss+xml" title="doesn&#039;t properly quote" href="https://bugs.horde.org/ticket/1146/rss" /> 
  <description>doesn&#039;t properly quote</description> 
 
   
   
  <item> 
   <title>If the entry has a comma in it (for example, lastname, first</title> 
   <description>If the entry has a comma in it (for example, lastname, firstname), it doesn&#039;t properly quote the string.  The following fixes this:







--- /tmp/ldap.php.org	2005-01-13 08:37:13.727402078 -0600

+++ ldap.php	2005-01-13 08:31:39.689948716 -0600

@@ -290,7 +290,11 @@

                 $newrdn = &#039;&#039;;

                 foreach ($this-&gt;_params[&#039;dn&#039;] as $param) {

                     if (isset($attributes[$param])) {

-                        $newrdn .= $param . &#039;=&#039; . $attributes[$param] . &#039;,&#039;;

+			if (strpos($attributes[$param],&#039;,&#039;) === false) {

+                            $newrdn .= $param . &#039;=&#039; . $attributes[$param] . &#039;,&#039;;

+			} else {

+			    $newrdn .= $param . &#039;=&quot;&#039; . $attributes[$param] . &#039;&quot;,&#039;;

+			}

                     }

                 }

                 $newrdn = substr($newrdn, 0, -1);

</description> 
   <pubDate>Thu, 13 Jan 2005 14:38:04 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/1146#t4463</link> 
  </item> 
   
  <item> 
   <title>Please test the following patch which implements RFC 1485 qu</title> 
   <description>Please test the following patch which implements RFC 1485 quoting of DNs in the LDAP driver, which should cover the case you found plus a bunch of others that we weren&#039;t properly catching before:



http://cvs.horde.org/diff.php/turba/lib/Driver/ldap.php?r1=1.56&amp;r2=1.57&amp;ty=u



If this works for you, I&#039;ll merge it into the next stable Turba release.</description> 
   <pubDate>Sat, 15 Jan 2005 20:31:57 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/1146#t4586</link> 
  </item> 
   
  <item> 
   <title>No feedback. I&#039;d say, merge it.</title> 
   <description>No feedback. I&#039;d say, merge it.</description> 
   <pubDate>Mon, 07 Feb 2005 10:38:14 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/1146#t5255</link> 
  </item> 
   
  <item> 
   <title>Merged.</title> 
   <description>Merged.</description> 
   <pubDate>Wed, 09 Feb 2005 03:40:36 +0000</pubDate> 
   <link>https://bugs.horde.org/ticket/1146#t5357</link> 
  </item> 
   
   
 
 </channel> 
</rss> 
