Open Bug 935671 Opened 12 years ago Updated 12 years ago

Security Review - TokenServerClient

Categories

(mozilla.org :: Security Assurance: Review Request, task)

Product:
mozilla.org
Component:
Security Assurance: Review Request
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Due Date:

People

(Reporter: dchanm+bugzilla, Assigned: dchanm+bugzilla)

References

(
URL
)

Details

(Whiteboard: [score=medium] u= c= p=1 s=sprint 6)

Filing a security review bug as per our process. This bug has already been assigned to sprint 2 as per curtisk comment in bug 799732. Our sprint 2 will take place during the week of 11-Nov until 22-Nov APIs are documented in this thread https://mail.mozilla.org/pipermail/sync-dev/2013-August/000392.html We should have sufficient documentation to start the review during sprint 2. I'll circle back with the services team if we need more
Whiteboard: [score=medium] u= c= p=1 s=ready → [score=medium] u= c= p=1 s=sprint 2
I just remembered that the code landed in m-c doesn't include all the tests. You can see JUnit 4 tests (that don't run on TBPL) at https://github.com/mozilla-services/android-sync/commits/a04639d
Whiteboard: [score=medium] u= c= p=1 s=sprint 2 → [score=medium] u= c= p=1 s=sprint 4
Whiteboard: [score=medium] u= c= p=1 s=sprint 4 → [score=medium] u= c= p=1 s=sprint 5
Whiteboard: [score=medium] u= c= p=1 s=sprint 5 → [score=medium] u= c= p=1 s=sprint 6
You need to log in before you can comment on or make changes to this bug.