Closed Bug 1799265 Opened 1 year ago Closed 1 year ago

Add a name, configurable timeout, and more logging to MerinoClient

Categories

(Firefox :: Address Bar, task, P1)

task

Tracking

()

RESOLVED FIXED
108 Branch
Tracking Status
firefox108 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

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.

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
Regressions: 1799582
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
Flags: qe-verify-
Flags: in-testsuite+
Blocks: 1815018
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: