Page MenuHomePhabricator

Quick/short security review of Extension:Sentry
Closed, ResolvedPublic

Description

The extension's code will change a lot before deployment to production, so it might not be worth a full on review now, but given that Sentry is to deal with stack traces, I figured a quick look is warranted.

Related Objects

StatusAssignedTask
OpenTgr
ResolvedTgr
ResolvedGilles
OpenNone
OpenNone
OpenTgr
OpenTgr
ResolvedTgr
Resolvedcsteipp
ResolvedTgr
OpenNone
ResolvedTgr
DeclinedTgr
DeclinedTgr
StalledTgr
ResolvedTgr
StalledTgr
ResolvedTgr
OpenTgr
ResolvedKrinkle
DeclinedNone
OpenTgr
ResolvedTgr
OpenTgr
OpenNone
InvalidNone
StalledTgr
ResolvedTgr
OpenNone
Resolvedjcrespo
ResolvedTgr

Event Timeline

greg created this task.Jan 13 2015, 4:19 PM
greg updated the task description. (Show Details)
greg raised the priority of this task from to Normal.
greg assigned this task to csteipp.
greg added subscribers: Aklapper, Tgr, He7d3r and 7 others.

I talked through the privacy aspects with Tgr, and as long as access to the collection server is restricted to start, we should be ok on that.

raven.js doesn't look likely to harm us. @Tgr, who on multimedia is responsible for making sure that any security fixes in upstream will get deployed into our environment?

Tgr added a comment.Feb 6 2015, 12:55 AM

@Tgr, who on multimedia is responsible for making sure that any security fixes in upstream will get deployed into our environment?

I'll assume responsibility for that.

csteipp closed this task as Resolved.Feb 6 2015, 12:57 AM
csteipp set Security to None.