[Regression] Since version 135.0 DevTools Network tab doesn't log requests if URL prefixed with view-source:
Categories
(DevTools :: Netmonitor, defect, P2)
Tracking
(firefox-esr115 unaffected, firefox-esr140 wontfix, firefox146 wontfix, firefox147 wontfix, firefox148 wontfix, firefox149 fixed)
People
(Reporter: a.broughton.en, Assigned: bomsy)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:146.0) Gecko/20100101 Firefox/146.0
Steps to reproduce:
- Open firefox to new tab
- Open DevTools (F12 or Right-Click Inspect in page or CTRL+SHIFT+I)
- Select Network tab
- Select Settings cog on right side and ensure Persist Logs is selected
- Visit URL:
view-source:https://bugzilla.mozilla.org
Regression in version 135.0, version 134.0.2 and before correctly log the requests in the network tab
| Version | Status |
|---|---|
| 134.0.2 (64-bit) | Shows network request - OK |
| 135.0 (64-bit) | Network request now shown - !OK |
Nightly 148.0a1 (20251217214341) tested, also faces regression.
This was discovered while tracking redirects from a malicious link, it redirected through three different domains and not a single requests was logged if the URL was prefixed with view-source:. Versions prior to 135.0 worked as intended / expected.
Actual results:
No requests listed in Network tab
Expected results:
Request should have been logged
| Reporter | ||
Updated•5 months ago
|
| Reporter | ||
Comment 1•5 months ago
|
||
Type in table: 135.0 Status should show not shown, instead of now shown.
| Version | Status |
|---|---|
| 134.0.2 (64-bit) | Shows network request - OK |
| 135.0 (64-bit) | Network request not shown - !OK |
| Assignee | ||
Updated•5 months ago
|
| Assignee | ||
Comment 2•5 months ago
|
||
Thanks for filing. Interesting. I wonder if this was affected by Bug 1932818
| Assignee | ||
Updated•5 months ago
|
| Reporter | ||
Comment 3•4 months ago
|
||
Is the needinfo and regressionwindow-wanted requesting information from me? Regression is since version 135.0. What other info is required?
Comment 4•4 months ago
|
||
(In reply to a.broughton.en from comment #3)
Is the needinfo and regressionwindow-wanted requesting information from me? Regression is since version 135.0. What other info is required?
the needinfo is for bomsy to look at, and the other flag was to run mozregression (https://mozilla.github.io/mozregression/) to get a shorter regression window, but that wasn't for you to do, you're all good :)
I was only able to get this pushlog https://hg-edge.mozilla.org/mozilla-central/pushloghtml?fromchange=b6fe6098a8bbf74edca23103c6756be13d238378&tochange=c9170b504cdce7fd51d1934bad32709bea49b112 , which is a bit large, but does include Bug 972821 and Bug 1932818 , which could impact this
| Assignee | ||
Comment 5•4 months ago
•
|
||
Hi a.broughton.en,
Just got back from holiday, yes it was as Nicholas said, the need-info was for me.
Thanks Nicholas for the regression window to confirm, i'll take a look to fix
Comment 6•4 months ago
|
||
P2 since it's a regression and bomsy already started investigating the issue.
Comment 7•4 months ago
|
||
Set release status flags based on info from the regressing bug 1932818
Updated•4 months ago
|
| Assignee | ||
Comment 8•4 months ago
|
||
Updated•4 months ago
|
| Assignee | ||
Updated•4 months ago
|
Updated•4 months ago
|
Comment 9•3 months ago
|
||
Hubert, are you going to land this patch before the merge?
Comment 10•3 months ago
|
||
| Assignee | ||
Comment 11•3 months ago
|
||
Hubert, are you going to land this patch before the merge?
Landed! Thanks for the reminder.
Comment 12•3 months ago
|
||
Comment 13•3 months ago
|
||
Reverted this because it was causing devtools failures in browser_resources_network_event_stacktraces_keepalive.js.
- Revert link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | devtools/shared/commands/resource/tests/browser_resources_network_event_stacktraces_keepalive.js | Stack trace for keepalive request was received
Please also check these failures.
| Assignee | ||
Comment 14•3 months ago
|
||
Thanks, i've fixed the issue and just pushed. to auto land.
Comment 15•3 months ago
|
||
Comment 16•3 months ago
|
||
| bugherder | ||
| Reporter | ||
Comment 17•3 months ago
|
||
I look forward to the release.
Thank you for your work on this! :)
Updated•2 months ago
|
Description
•