Closed
Bug 1582379
Opened 5 years ago
Closed 5 years ago
Include more context in Rust mozStorage errors
Categories
(Toolkit :: Storage, task, P1)
Toolkit
Storage
Tracking
()
RESOLVED
FIXED
mozilla71
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: lina, Assigned: lina)
References
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•5 years ago
|
||
A low-frequency error that showed up in telemetry for new bookmark
sync is "Storage operation failed with NS_ERROR_CANNOT_CONVERT_DATA",
which occurs when we try to convert a variant to the wrong type. It
would be helpful to include the type and column names for this case;
for example, "Can't get i32 for column lastModified". Neither the type
nor column name are PII; we want them to show up in telemetry.
Assignee | ||
Comment 2•5 years ago
|
||
I don't think we'll need to uplift this patch, but it would be good to get it landed soon, so we can see the error in Nightly, land, and uplift that fix.
Pushed by kcambridge@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5f81726e0ea9 Include more context in Rust mozStorage errors. r=tcsc
Comment 4•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in
before you can comment on or make changes to this bug.
Description
•