Closed
Bug 2030155
Opened 1 month ago
Closed 1 month ago
String should deserialize without quotes
Categories
(Firefox for Android :: Experimentation and Telemetry, task)
Tracking
()
RESOLVED
FIXED
151 Branch
| Tracking | Status | |
|---|---|---|
| firefox151 | --- | fixed |
People
(Reporter: olivia, Assigned: olivia)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][group6])
Attachments
(1 file)
Confirm we aren't sending string prefs values with quotes "".
My branch was in an odd state when testing, but when testing some with D292433 I saw a string pref be comited as "my-value" instead of my-value.
Updated•1 month ago
|
| Assignee | ||
Updated•1 month ago
|
Summary: Confirm string is serializing without quotes → String should serializing without quotes
| Assignee | ||
Updated•1 month ago
|
Summary: String should serializing without quotes → String should deserialize without quotes
| Assignee | ||
Comment 1•1 month ago
|
||
Right now, the app isn't properly parsing strings coming from Nimbus.
For example, the Nimbus Gecko pref String value of hello-world is literally
posting as "hello-world" with quotes instead of hello-world.
This patch adds handling to account for JSON string values and raw values.
Pushed by ohall@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/2bcce7fc15cf
https://hg.mozilla.org/integration/autoland/rev/09859f4d991e
JSON Parse String Values from Nimbus r=android-reviewers,mcarare
Comment 3•1 month ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 1 month ago
status-firefox151:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•