[MAILBOX-393] MailboxManager::deleteMailbox by mailboxId

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.5.0
• Component/s: api
• Assignee: Unassigned
• Reporter: Benoit Tellier
• Created: 06/Dec/19 04:36
• Updated: 13/Dec/19 10:02
• Resolved: 13/Dec/19 10:02

Description

Currently we can only delete a mailbox by it's mailboxPath.

MailboxPath is mutable as a mailbox can be renamed.

Mutable identifiers are bad, and thus race conditions can arise upon mailbox deletion:

Given a mailboxId
I read the corresponding mailboxPath
and a concurrent session renames the mailbox (switch name) after my read
When I delete the mailbox using the previously read path
Then I bloew up the wrong mailbox (!!!)