Closed
Bug 1799265
Opened 2 years ago
Closed 2 years ago
Add a name, configurable timeout, and more logging to MerinoClient
Categories
(Firefox :: Address Bar, task, P1)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
108 Branch
Tracking | Status | |
---|---|---|
firefox108 | --- | fixed |
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
The new weather feature for quick suggest will use its own instance of MerinoClient and a different timeout from the usual one stored in the pref. Adding a name to MerinoClient would make it easier to understand logging. MerinoClient could also use more logging too.
Assignee | ||
Comment 1•2 years ago
|
||
This adds a new name
property to MerinoClient that's included in log
messages. That makes it easier to understand logs when there's more than one
client, as there will be since the new weather feature will use its own. This
also adds more logging.
It also adds a timeout param to fetch()
. The param overrides the timeout pref.
The weather MerinoClient will use a custom timeout.
Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bea4644a77ba
Add a name, configurable timeout, and more logging to MerinoClient. r=daisuke
Comment 3•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox108:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
Assignee | ||
Updated•2 years ago
|
Flags: qe-verify-
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•