Tickets :: [#9299] quoting of folder names when setting ACLs is broken

6.0.0-beta1

8/13/25

Summary	quoting of folder names when setting ACLs is broken
Queue	Horde Framework Packages
Queue Version	FRAMEWORK_3
Type	Bug
State	Resolved
Priority	2. Medium
Owners	slusarz (at) horde (dot) org
Requester	Graeme.Wood (at) ed (dot) ac (dot) uk
Created	10/07/2010 (5424 days ago)
Due
Updated	10/20/2010 (5411 days ago)
Assigned	10/07/2010 (5424 days ago)
Resolved	10/20/2010 (5411 days ago)
Github Issue Link
Github Pull Request
Milestone	3.3.10
Patch	No

10/20/2010 04:12:34 PM	Michael Slusarz	Comment #8 State ⇒ Resolved	Reply to this comment
I have no idea about the delay (I didn't originally write any of this ACL code, unfortunately). I see there is a reference to Cyrus Murder and a delay. Is that what you are using? At best, though, that is a separate issue. This issue has been resolved.

10/20/2010 10:41:57 AM	Jan Schneider	Summary ⇒ quoting of folder names when setting ACLs is broken

10/20/2010 10:14:12 AM	Graeme (dot) Wood (at) ed (dot) ac (dot) uk	Comment #7	Reply to this comment
Almost. It does set ACLs however there looks like there is a timing issue, as it returns a message to say it could not retrieve the ACL after it set it. If you then go and select the folder again and retrieve the ACL, it correctly displays.

10/19/2010 06:15:14 PM	Michael Slusarz	Comment #6 Summary ⇒ quoting of folder names when setting ACLs is brokenC	Reply to this comment
Corrected links: http://cvs.horde.org/diff.php/framework/IMAP/IMAP/ACL/Attic/rfc2086.php?rt=horde&r1=1.6.8.26&r2=1.6.8.27&ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.515.2.629&r2=1.515.2.630&ty=u

10/19/2010 06:13:55 PM	Michael Slusarz	Comment #5 Assigned to Michael Slusarz Priority ⇒ 2. Medium Milestone ⇒ 3.3.10	Reply to this comment
Does this fix work?

10/19/2010 06:12:31 PM	CVS Commit	Comment #4	Reply to this comment
Changes have been made in CVS for this ticket: ~~Bug: 9299~~ Fix accessing IMAP ACLs that contain non-alphanumeric characters. imap_*() methods don't require quoting of the mailbox name. If we directly talk to IMAP server, always quote mailbox name (this is not 100% correct IMAP behavior either, but it is as good as we are going to get in this library for Horde 3). http://cvs.horde.org/diff.php/framework/IMAP/IMAP/ACL/Attic/rfc2086.php?rt=horder1=1.6.8.26r2=1.6.8.27ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horder1=1.515.2.629r2=1.515.2.630ty=u

10/07/2010 03:59:42 PM	Graeme (dot) Wood (at) ed (dot) ac (dot) uk	Comment #3	Reply to this comment
Possibly a duplicate of ~~Bug #9058~~? No it isn't. You can sort of get it working by hacking out the quoting code and then it will sort of work for folders with hyphens and underscores, so it has nothing to do with capability.

10/07/2010 03:53:00 PM	Michael Slusarz	Comment #2 State ⇒ Feedback	Reply to this comment
Possibly a duplicate of ~~Bug #9058~~?

10/07/2010 03:10:42 PM	Graeme (dot) Wood (at) ed (dot) ac (dot) uk	Comment #1 State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Horde Framework Packages Due ⇒ 10/07/2010 Summary ⇒ quoting of folder names when setting ACLs is broken Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
The upgrade to v3.3.9 of Horde has broken the setting of ACLs on folders with non-alphanumeric characters in their names. The ACL driver returns: The current list of users with access to this folder could not be retrieved. lib/Horde/IMAP/ACL/rfc2086.php has changed to use imap_setacl and imap_getacl instead of talking direct to the IMAP server. Something about quoting folder names is broken. Putting the older rfc2086.php from 3.3.8 back restores the function.