Closed Bug 1147275 Opened 9 years ago Closed 9 years ago

Delete cached oauth tokens when Firefox Account is deleted

Categories

(Android Background Services Graveyard :: Reading List Sync, defect)

All
Android
defect
Not set
normal

Tracking

(firefox40 fixed)

RESOLVED FIXED
Firefox 40
Tracking Status
firefox40 --- fixed

People

(Reporter: nalexander, Assigned: vivek, Mentored)

References

Details

Attachments

(1 file, 1 obsolete file)

57 bytes, text/x-github-pull-request
nalexander
: review+
Details | Review
(In reply to Nick Alexander :nalexander from comment #1)
> https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/fxa/
> receivers/FxAccountDeletedService.java#69

Could do delete tokens should go around here.
Attached file android-sync PR (obsolete) —
Attachment #8588686 - Flags: review?(nalexander)
Comment on attachment 8588686 [details] [review]
android-sync PR

Updated patch, one nit, and I think we're good.
Attachment #8588686 - Flags: review?(nalexander) → feedback+
Attached file android-sync PR
PR updated with review comments
Attachment #8588686 - Attachment is obsolete: true
Attachment #8590556 - Flags: review?(nalexander)
Assignee: nobody → vivekb.balakrishnan
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/82c4a7172cbe
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Comment on attachment 8590556 [details] [review]
android-sync PR

Landed.  Thanks vivek!
Attachment #8590556 - Flags: review?(nalexander) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: