1636804 - Network monitor now wrapping all values of HTTP headers with double quotes, leading to awkward escaping of values that actually contain quotes

Status: RESOLVED FIXED

(DevTools :: Netmonitor, defect, P2)

Description

[Jake Jarvis]

Attachment: parsed-vs-raw.png

All header values in the network monitor are now surrounded by double quotes, straying from how they're shown in the raw data view (and everywhere else — curl -I, Chrome devtools, etc). I'm on 77.0b3 and it seems like this also popped up and was fixed in Bug 1258628 four years ago.

I (very quickly) skimmed the diff of Bug 1613884 where it looks like this changed and based on the tests here (L62 & L64 of test/browser_net_post-data-03.js) it seems like this might be expected behavior now:

https://phabricator.services.mozilla.com/differential/changeset/?ref=2404029

...but as seen in the screenshots, this gets really awkward-looking when it comes to headers that actually have double quotes in them (in my humble opinion, at least!). ETags are a good ubiquitous example of this.

Feel free to completely disregard if this is truly intentional, it just caught me off-guard today and after finding Bug 1258628 I thought it was worth reporting. :)

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1613884

Comment 2

[:Harald Kirschner :digitarald]

Bomsy, could we drop the quotes and escaping altogether, as the JSON string treatment doesn't make much sense for headers where everything is a string? This would reduce dead space and simplify the UI.

P2, as this also adds " to all values copied by text selection.

Comment 3

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1636804 - Remove quotes and escaping for header sections r=nchevobbe

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/be73bced2080
Remove quotes and escaping for header sections r=nchevobbe

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/be73bced2080

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Should we nominate this for Beta approval to get it fixed in 77 before it ships to release? Please do so ASAP if yes.

Comment 7

[Hubert Boma Manilla (:bomsy)]

Comment on attachment 9150400 [details]
Bug 1636804 - Remove quotes and escaping for header sections r=nchevobbe

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Affects only web developers, its a simple change, just changes property values
• User impact if declined: Affects only web developers
• Fix Landed on Version: 77
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Simple change, changing property values
• String or UUID changes made by this patch:

Comment 8

[Pascal Chevrel:pascalc]

Comment on attachment 9150400 [details]
Bug 1636804 - Remove quotes and escaping for header sections r=nchevobbe

The patch looks safe, has tests and is a 77 regression. Uplift approved for beta. Please be cautious with the branch you request for uplift (you requested the es branch and it was meant for beta, almost missed it, thanks jcristau for noticing!)

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/63896e87d70e

Comment 10

[Hubert Boma Manilla (:bomsy)]

Ahh .. my bad thanks jcristau and pascal