Tickets :: [#5765] Accept various newlines in LDIF import

6.0.0-beta1

7/23/25

Summary	Accept various newlines in LDIF import
Queue	Turba
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	Horde Developers (at) , selsky (at) columbia (dot) edu
Requester	gregory (at) langenburgresearch (dot) org
Created	10/03/2007 (6503 days ago)
Due
Updated	10/03/2007 (6503 days ago)
Assigned
Resolved	10/03/2007 (6503 days ago)
Milestone
Patch	No

10/03/2007 07:48:25 PM	Matt Selsky	Comment #5 State ⇒ Resolved	Reply to this comment
I added a test for this: http://cvs.horde.org/co.php/turba/lib/tests/ldif_importFileCRLF.phpt?r=1.1

10/03/2007 07:27:32 PM	Matt Selsky	Comment #4 Assigned to Matt Selsky Assigned to Horde Developers State ⇒ Feedback	Reply to this comment
Try this patch: http://lists.horde.org/archives/cvs/Week-of-Mon-20071001/071126.html

10/03/2007 07:17:35 PM	gregory (at) langenburgresearch (dot) org	Comment #3 New Attachment: ldif.php.patch	Reply to this comment
Some clients export LDIF with windows newlines, attatched patch allows import of these files. RFC 2849 defines SEP: SEP = (CR LF / LF) Are you seeing something else? aka lone CR characters? I just put those in there 'for good measure'. Conforming to RFC, and matching only CRLF and LF should be fine. Attatched new patch.

10/03/2007 06:44:01 PM	Matt Selsky	Comment #2	Reply to this comment
Some clients export LDIF with windows newlines, attatched patch allows import of these files. RFC 2849 defines SEP: SEP = (CR LF / LF) Are you seeing something else? aka lone CR characters?

10/03/2007 05:02:18 PM	gregory (at) langenburgresearch (dot) org	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Accept various newlines in LDIF import Queue ⇒ Turba New Attachment: newlines.patch State ⇒ New	Reply to this comment
Some clients export LDIF with windows newlines, attatched patch allows import of these files.