New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 611960 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 611905



Sign in to add a comment

//services/catalog/public/interfaces/catalog.mojom needs a security review

Project Member Reported by dcheng@chromium.org, May 14 2016

Issue description

These interfaces don't appear to have gone through security review. Marking ReleaseBlock-Stable to be on the safe side for now.

 

Comment 1 by dcheng@chromium.org, May 14 2016

Blocking: 611905
Labels: OS-All
[Bulk edit]

Release blockers need OS set.  Looks like all of these are applicable for OS-All, correct if required.
This issue is set as M52 Release block Stable. Is there anything which needs to to be taken care in M52? Please resolve ASAP.
Owner: dcheng@chromium.org
Status: Assigned (was: Available)
Assigning to Daniel as per offline conversation.Work is in progress and ship with M52 Stable.

M52 Pre-stable promotion is planned during July 2-nd week, please plan accordingly.

Comment 5 Deleted

dcheng@ : Gentle ping to merge your changes to M52 branch at the earliest.

Comment 7 Deleted

Comment 8 by gov...@chromium.org, Jun 27 2016

** IMPORTANT change in M52 merge date due to first 2 weeks of July no release weeks **
M52 Stable is launching very soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged ASAP. All changes MUST be merged into the release branch by 5pm on July 1 to make into the desktop Stable final build cut. Thank you!

Comment 9 by dcheng@chromium.org, Jun 29 2016

Labels: -ReleaseBlock-Stable
Project Member

Comment 10 by sheriffbot@chromium.org, Jun 30 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Te-NeedsFurtherTriage
Adding TE-NeedsFurtherTriage as it can't be triaged by TE. 
Cc: dcheng@chromium.org
Owner: palmer@chromium.org
Project Member

Comment 13 by sheriffbot@chromium.org, Jul 10 2016

Labels: -M-53 -Pri-1 M-54 MovedFrom-53 Pri-2
This issue is Pri-1 but has already been moved once. Lowering the priority and moving to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: Internals>Mojo
Components: Security>Audit
Removing from the security sheriff queue.
Labels: -TE-NeedsfurtherTriage TE-NeedsTriageHelp
Cc: tsepez@chromium.org
tsepez: Any chance you're interested in picking this up?
Status: Fixed (was: Assigned)
Looking at the straightforward implementation of these interfaces, I don't see any problems.

Sign in to add a comment