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 (
As we discussed in Standup you and I have the same question here in terms of
how wide do we want to open this and if we should inherit permissions from Bug
(where owner is not really an owner), or CVE (which doesn't even have an owner
at all)...
> + 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
