Include URL parameters in network request logs

VERIFIED FIXED in Firefox 60

Status

VERIFIED FIXED
2 years ago
5 months ago

People

(Reporter: sebo, Assigned: abhinav.koppula, Mentored)

Tracking

(Blocks: 1 bug, {good-first-bug})

53 Branch
Firefox 60
good-first-bug

Firefox Tracking Flags

(firefox60 verified)

Details

Attachments

(1 attachment, 2 obsolete attachments)

59 bytes, text/x-review-board-request
Honza
: review+
Details
(Reporter)

Description

2 years ago
Network requests logged to the Web Console are missing URL parameters.
This requires a user to expand the requests or switch to the Network Monitor, if the same URL path is called with different parameters.

FWIW, in Firebug the URLs included the parameters and were truncated in case they are too long.

Sebastian
(Reporter)

Comment 1

2 years ago
This feature may be optional (i.e. behind a preference) in case there are concerns that long URLs may cause UI issues.

Additional note about Firebug's implementation: It showed the full URL on hover.

Sebastian
(Reporter)

Comment 2

2 years ago
Btw. this was asked for at http://stackoverflow.com/q/41945654/432681.

Sebastian
Instead of adding a preference for these things, I'd prefer if they were implemented naturally. A good solution would be doing better cropping (i.e. at the center), so we can have both the end, and the start of the URL. Or maybe a start cropping would be better.
Also, the new console gets rid of the long status code at the end which definitively helps.
(Reporter)

Comment 5

2 years ago
(In reply to Tim Nguyen :ntim from comment #3)
> Instead of adding a preference for these things, I'd prefer if they were
> implemented naturally. A good solution would be doing better cropping (i.e.
> at the center), so we can have both the end, and the start of the URL. Or
> maybe a start cropping would be better.

Side note: Cropping in the middle of a string via CSS is not possible yet. See bug 740910.

Sebastian
(Reporter)

Comment 6

a year ago
I assume this shouldn't be too hard to implement, so I add the 'good-first-bug' keyword.

Sebastian
Keywords: good-first-bug
See Also: → bug 740910

Comment 7

a year ago
Currently the query params are removed here
> request.url.replace(/\?.+/, "")
By simply removing the replace statement, the query params would be viewable, however long urls will be trimmed by the standard styling. Currently the expanded view of the line contains a params tab with any included params. While it is possible to have the beginning of the request truncated, this does not guarantee that all query params would be viewable.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 11

11 months ago
Hi Sebastian,
Is this issue still open for anyone to take?
Flags: needinfo?(sebastianzartner)
(Reporter)

Comment 12

11 months ago
I redirect that question to Robert, as he already started working on it (without being assigned).

Sebastian
Flags: needinfo?(sebastianzartner) → needinfo?(r.f.masen)

Comment 13

11 months ago
Abhinav,
  Feel free to work on this issue. I submitted my pull request 4 months ago and it hasen't been reviewed. I am not sure I understand how this whole process is supposed to work but I am not sure when I will have time again to figure it out.
Flags: needinfo?(r.f.masen)
(Reporter)

Updated

11 months ago
Attachment #8906107 - Flags: review?(odvarko)
(Reporter)

Updated

11 months ago
Attachment #8906111 - Flags: review?(odvarko)
(Reporter)

Comment 14

11 months ago
Thank you for your answer, Robert! The normal process is to ask someone specific for review.
I think in this case it might be Honza, so I've set him as reviewer and mentor for this bug.
@Honza Please redirect the review if you can't review this patch.

Sebastian
Assignee: nobody → abhinav.koppula
Mentor: odvarko
Status: NEW → ASSIGNED
I like the idea of having URL params displayed by default.

Harald, what do you think?

Honza
Flags: needinfo?(hkirschner)
I like it as it is one of the essential information that makes up a request (URL, method & params)
Flags: needinfo?(hkirschner)

Comment 17

11 months ago
mozreview-review
Comment on attachment 8906107 [details]
Bug 1335316

https://reviewboard.mozilla.org/r/177852/#review215706

Please rebase on the latest HEAD (the file has been renamed)

Thanks!
Honza
Attachment #8906107 - Flags: review?(odvarko) → review-

Comment 18

11 months ago
mozreview-review
Comment on attachment 8906111 [details]
not sure how that line got deleted... Bug 1335316

https://reviewboard.mozilla.org/r/177860/#review215708

Please merge this patch into the first one.
Honza
Attachment #8906111 - Flags: review?(odvarko) → review-
(Reporter)

Comment 19

10 months ago
Abhinav, are you still interested in finishing up the patch for this?

Sebastian
Flags: needinfo?(abhinav.koppula)
Comment hidden (mozreview-request)
(Assignee)

Comment 21

10 months ago
Hi Sebastian,
Sorry for the delay. Have rebased the patch.
Flags: needinfo?(abhinav.koppula)

Comment 22

10 months ago
mozreview-review
Comment on attachment 8944033 [details]
Bug 1335316 - Include URL parameters in network request logs;

https://reviewboard.mozilla.org/r/214356/#review220154

Thanks!

R+ assuming try is green

Honza
Attachment #8944033 - Flags: review?(odvarko) → review+
Attachment #8906107 - Attachment is obsolete: true
Attachment #8906111 - Attachment is obsolete: true
(Assignee)

Updated

10 months ago
Keywords: checkin-needed

Comment 24

10 months ago
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d74298d2f241
Include URL parameters in network request logs; r=Honza
Keywords: checkin-needed

Comment 25

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d74298d2f241
Status: ASSIGNED → RESOLVED
Last Resolved: 10 months ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60

Comment 26

9 months ago
I Have tried to Reproduce the Same Bug in Firefox Nightly  60.0a1 on Windows 10 (64bit) & Windows 7 (32bit)

------------------------------------------------------------------------------------------
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:60.0) Gecko/20100101 Firefox/60.0
Build ID: 20180212100154
Firefox Version: 60.0a1
------------------------------------------------------------------------------------------
User Agent: Mozilla/5.0 (Windows NT 6.1; rv:60.0) Gecko/20100101 Firefox/60.0
Build ID: 20180211220056
Firefox Version: 60.0a1
------------------------------------------------------------------------------------------

But the Bug has been Fixed. 

[Steps to Reproduce]

1) Started the Firefox with Cleaned Profile
2) Then opened this URL https://bugzilla.mozilla.org/show_bug.cgi?id=1335316 in a New tab or Try any other URLs too.
3) Press the Ctrl+Shift+E for Network Monitor 
4) To Perform a request or "Reload" the same page to see detailed information about network activity.

Actual Results:
URL parameters Displayed in the requests logged to the console.

Status: Fixed & Verified 

[bugday-20180214]
(Reporter)

Updated

9 months ago
Status: RESOLVED → VERIFIED
status-firefox60: fixed → verified

Updated

5 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.