Diff comments:
> diff --git a/lib/lp/security.py b/lib/lp/security.py
> index 22bc140..98e6f26 100644
> --- a/lib/lp/security.py
> +++ b/lib/lp/security.py
> @@ -3781,3 +3782,12 @@ class EditCIBuild(AdminByBuilddAdmin):
> if auth_repository.checkAuthenticated(user):
> return True
> return super().checkAuthenticated(user)
> +
> +
> +class EditVulnerability(AuthorizationBase):
> + permission = 'launchpad.Edit'
> + usedfor = IVulnerability
> +
> + def checkAuthenticated(self, user):
> + return (
Do we want to allow roles other than admins and commercial admins to edit a
vulnerability? I am not sure that team members from the security team, who will
be using this, have admin roles.
> + user.in_commercial_admin or user.in_admin)
--
https://code.launchpad.net/~ilasc/launchpad/+git/launchpad/+merge/415966
Your team Launchpad code reviewers is requested to review the proposed merge of
~ilasc/launchpad:add-vulnerability-orm into launchpad:master.
_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help : https://help.launchpad.net/ListHelp
