Closed Bug 1073047 Opened 5 years ago Closed 5 years ago

Add Firefox Account logging to MozLoopService

Categories

(Hello (Loop) :: Client, enhancement)

enhancement
Not set
Points:
1

Tracking

(firefox34 fixed, firefox35 fixed)

RESOLVED FIXED
mozilla35
Iteration:
35.2
Tracking Status
firefox34 --- fixed
firefox35 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

(Whiteboard: [loop-uplift])

Attachments

(1 file)

This adds debug logging for Firefox Account integration points in MozLoopService. 

I didn't add logs around every hawk request since it already has its own logging pref (services.hawk.loglevel) likewise around some FxA modules (that log with identity.fxaccounts.loglevel). We can always add more later in other bugs when we see something not covered.
Attachment #8495368 - Flags: review?(jaws)
Flags: qe-verify-
Flags: firefox-backlog+
https://hg.mozilla.org/integration/fx-team/rev/06a1582f7d5f

Thanks
Flags: in-testsuite-
Whiteboard: [loop-uplift] → [loop-uplift][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/06a1582f7d5f
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Whiteboard: [loop-uplift][fixed-in-fx-team] → [loop-uplift]
Target Milestone: --- → mozilla35
Comment on attachment 8495368 [details] [diff] [review]
v.1 log.debug in some areas not covered by Hawk/FxA module logs

Approval Request Comment
Uplift request for patches staged and tested on Fig
Attachment #8495368 - Flags: approval-mozilla-aurora?
Comment on attachment 8495368 [details] [diff] [review]
v.1 log.debug in some areas not covered by Hawk/FxA module logs

I worked with Randell and Maire on uplifting a large number of Loop bugs at once. All of the bugs have been staged on Fig and tested by QE before uplift to Aurora. As well, all of the bugs are isolated to the Loop client. Randell handled the uplift with my approval. I am adding approval to the bug after the fact for bookkeeping.
Attachment #8495368 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.