6.0.0-beta1
7/3/25

[#14732] If trying to delete a default system folder (Contacts, Calendar, Drafts, ...), FolderSync should return status 3 and not status 6
Summary If trying to delete a default system folder (Contacts, Calendar, Drafts, ...), FolderSync should return status 3 and not status 6
Queue Synchronization
Queue Version FRAMEWORK_5_2
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester john.bieling (at) gmx (dot) de
Created 11/10/2017 (2792 days ago)
Due
Updated 11/13/2017 (2789 days ago)
Assigned 11/10/2017 (2792 days ago)
Resolved 11/13/2017 (2789 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
11/13/2017 08:31:03 AM john (dot) bieling (at) gmx (dot) de Comment #8 Reply to this comment
Hi,

thanks for your work! The client I am using is "TbSync" for 
Thunderbird. We have talked about it a few months ago.

In the final version I also will not allow the user to delete default 
system folders, but in the current state I want to get the ActiveSync 
implementation and the error handling right, so I am actually fuzzing, 
to see, if TbSync can recover from all sorts of errors.

I am testing against a Horde 5.2 server from one of my users (who 
reported the "there is no policy for this device" bug to me), but I do 
not have access to the latest dev branch, so I cannot test your fixes. 
But you could try the latest Version of TbSync:

https://github.com/jobisoft/TbSync/releases/download/v0.6.8.2/TbSync-0.6.8.3-13.11.2017.xpi

If you could provide login data (via email) to an account on a Horde 
server which includes these fixes, I can of course do the test by 
myself.

TbSync had some UI problem with Thunderbird for Apple Mac, which I 
could not work on until now, because I do not have one. Maybe it is 
solved by now, but I cannot tell. Linux and Windows are working fine.

Thanks
John

There is a UI bug on Apple MAC which I could not work on, but maybe it 
works now?
...and since I botched the ticket number in the commit message, here 
are the commits containing the fixes.

https://github.com/horde/Core/commit/90e9883c3401a7038f4addc44c9fbe99d0e816e4

https://github.com/horde/ActiveSync/commit/a6301fe6d2f0bc7962be4f629ed59c1a5d9cef65
11/13/2017 12:07:31 AM Michael Rubinsky Comment #7 Reply to this comment
...and since I botched the ticket number in the commit message, here 
are the commits containing the fixes.

https://github.com/horde/Core/commit/90e9883c3401a7038f4addc44c9fbe99d0e816e4

https://github.com/horde/ActiveSync/commit/a6301fe6d2f0bc7962be4f629ed59c1a5d9cef65
11/13/2017 12:05:32 AM Michael Rubinsky Comment #6
State ⇒ Resolved
Reply to this comment
Fixed in both Horde_ActiveSync and Horde_Core.

It would be great if you could test these changes before release, as I 
can't find a client that allows attempting to delete a known special 
mailbox folder.
11/13/2017 12:03:18 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit b6f179364a1151c94c579ee88945c5ee5e18ef86
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sun, 12 Nov 2017 19:01:13 -0500

[mjr] Detect if the failure was due to trying to delete a special 
mailbox (Bug #14732).

  M doc/Horde/Core/CHANGES
  M package.xml

https://github.com/horde/Core/commit/b6f179364a1151c94c579ee88945c5ee5e18ef86
11/13/2017 12:03:17 AM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit c042a52419854f6765fcaf19a5b3be4d1b9ea5b8
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sun, 12 Nov 2017 19:01:13 -0500

[mjr] Detect if the failure was due to trying to delete a special 
mailbox (Bug #14732).

  M doc/Horde/Core/changelog.yml

https://github.com/horde/Core/commit/c042a52419854f6765fcaf19a5b3be4d1b9ea5b8
11/13/2017 12:03:03 AM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 67fd551ade977be944de37d683d492242a27ea5e
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sun, 12 Nov 2017 19:02:26 -0500

[mjr] Detect if the failure was due to trying to delete a special 
mailbox (Bug #14732).

  M doc/Horde/Core/changelog.yml

https://github.com/horde/Core/commit/67fd551ade977be944de37d683d492242a27ea5e
11/12/2017 11:43:15 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 1e461c9df2c619156fd1d49b406b00162d319c4b
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sun, 12 Nov 2017 18:42:52 -0500

Detect if the failure was due to trying to delete a special mailbox.

Part of the fix for Bug: 14732

  M lib/Horde/Core/ActiveSync/Driver.php

https://github.com/horde/Core/commit/1e461c9df2c619156fd1d49b406b00162d319c4b
11/10/2017 08:49:24 PM Jan Schneider Assigned to Michael Rubinsky
State ⇒ Assigned
 
11/10/2017 08:40:03 AM john (dot) bieling (at) gmx (dot) de Comment #1
Milestone ⇒
State ⇒ Unconfirmed
Patch ⇒ No
Queue ⇒ Synchronization
Summary ⇒ If trying to delete a default system folder (Contacts, Calendar, Drafts, ...), FolderSync should return status 3 and not status 6
Type ⇒ Bug
Priority ⇒ 1. Low
Reply to this comment
Status 3: Dedicated status if a system folder was requested to be deleted
Status 6: General error on server

Saved Queries