Comment 2 for bug 1720182

Revision history for this message
Colin Watson (cjwatson) wrote :

I don't think it would be a particularly large overhaul at all. xPPH.requestDeletion is an odd special case that goes through AppendArchive, which is otherwise only used by a few weird and/or legacy methods (Archive.syncSource[s], Archive.newSubscription, and Archive.removeCopyNotification). It would be quite easy to get it to check Archive.canAdministerQueue instead, if Ubuntu would be OK with that from a policy perspective. Doing that wouldn't affect any other methods.

The name of the "queue admin" archive permission type would indeed become a slight misnomer if we did that, but it's already called just "admin" in the edit-acl client tool, and I reckon the retcon of saying that it actually means admin privileges over the suite rather than just the queue associated with the suite would be a reasonable one to perform.