Closed Bug 1715988 Opened 4 years ago Closed 4 years ago

Filtering with incomplete status-code should treat the filter input as "includes", not "equals"

Categories

(DevTools :: Netmonitor, enhancement, P3)

enhancement

Tracking

(firefox91 fixed)

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: nchevobbe, Assigned: claubatista, Mentored)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

Steps to reproduce

  1. Open a data:text/html,<meta charset=utf8> tab
  2. Open devtools, select the netmonitor, and open the split console
  3. Evaluate
fetch("https://httpstat.us/400")
fetch("https://httpstat.us/403")
fetch("https://httpstat.us/404")

to populate the netmonitor

  1. In the netmonitor filter input, type status-code:4

Expected results

All the request with status code starting with 4 are displayed (so all the 3 network calls here)

Actual results

No requests are displayed


I think this is where the filtering happens: https://searchfox.org/mozilla-central/rev/af8e5d37fd56be90ccddae2203e7b875d3f3ae87/devtools/client/netmonitor/src/utils/filter-text-utils.js#146

So instead of having the equal, we could use a startsWith call

Thanks for reporting!
Looks like a good first bug.

Severity: -- → S3
Keywords: good-first-bug
Priority: -- → P3

Hi! I was able to reproduce the crash.
I'd like to work on this bug. :)

Go ahead!
Assigned to you!

Thanks.

Assignee: nobody → contatodaclau
Mentor: hmanilla
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/24f721024e86 [devtools] Match incomplete search terms in status-code:search. r=nchevobbe
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: