Skip to content
This repository was archived by the owner on Jul 10, 2024. It is now read-only.

Allow senior staff and managers to edit Bugzilla components #7

Open
emceeaich wants to merge 1 commit intomasterfrom
staff-edit-components
Open

Allow senior staff and managers to edit Bugzilla components #7
emceeaich wants to merge 1 commit intomasterfrom
staff-edit-components

Conversation

@emceeaich
Copy link
Contributor

RFC to distribute Bugzilla administration duties

RFC to distribute Bugzilla administration duties
@emceeaich
Copy link
Contributor Author

Issue #6

@emceeaich emceeaich changed the title Create staff-edit-components.md Allow senior staff and managers to edit Bugzilla components Feb 14, 2020
@emceeaich
Copy link
Contributor Author

Suggestion: if a member of a magic group posts a bug to a particular component, could we mint and send a one-time capability URL to the reporter which would let them make the change?

That way we'd have the public record of the change in the bug, and non-admin staff don't have long-lived permissions.

@mitchhentges
Copy link
Contributor

Hmm, that's interesting 🤔

and non-admin staff don't have long-lived permissions.

Since the non-admin staff are part of that magic group, wouldn't that count as still having a long-lived permission? It sounds like it's like a "permission proxy" such that the non-admins still have the ability to make changes, they just require one extra step (creating the bug) to perform the operation

@emceeaich
Copy link
Contributor Author

emceeaich commented Feb 14, 2020 via email

@mitchhentges
Copy link
Contributor

The additional step of requesting the capability would mitigate the harm they could do.

I don't quite understand this - the thing that sends a one-time capability URL sends it instantly, right?
If I'm grokking this correctly, this will mitigate harm by adding busywork to making changes. If someone is actively malicious, they could script it (even if we added timeouts to the capability-URL logic, it'd be a bit of a cat'n'mouse game).


Stepping back a bit here, my guess is that the core issue here is that we want to restrict the amount of people that have access to the editcomponents permission. At the same time, we don't want to be the only people that can edit components since that forces us to keep doing admin busywork for all projects.

Perhaps this will be nicely resolved by the "Extend self-service capabilities" roadmap item? It sounds like that means there will be a sort of "scoped editcomponents" permission that will allow product admins to have control over just their projects. Will "Extend self-service capabilities" make the need for this "permission proxy" obsolete?

@emceeaich
Copy link
Contributor Author

I don't quite understand this - the thing that sends a one-time capability URL sends it instantly, right?

It doesn't have to. It could be sent to the Bugmail address of the user, or indirectly accessed through the users' profile.

It sounds like that means there will be a sort of "scoped editcomponents" permission that will allow product admins to have control over just their projects.

That would obviate most of the need for this.

@emceeaich
Copy link
Contributor Author

emceeaich commented Feb 14, 2020

To start winding up the discussion, I'd like to allow a pilot group of users to have editcomponents through a group membership so we start reducing admin involvement, and build understanding of what "extending self-service capabilities" will entail.

@globau
Copy link

globau commented Feb 20, 2020

We should ensure we have reasonable step-by-step documentation for how to make common changes.

# Summary

Grant `editcomponents` to staff engineers, senior engineering manager in Bugzilla.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can grant editcomponents on a per-product basis using the group access controls UI of a product page.

https://bugzilla.mozilla.org/editproducts.cgi?action=editgroupcontrols&product=Core

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants