Skip to content

Fix querystring all items loop

Daniel Tian requested to merge 376324-fix-querystring-all-items-loop into master

What does this MR do and why?

This MR fixes an issue where if a filter on the vulnerability report is fixing bad IDs on the querystring, it will push a new history state that makes it impossible to go back to the previous page using the browser's back button unless the user mashes it or purposefully skips a state. It's hard to explain the issue in text form, so here's 2 videos detailing the issue and the fix:

The problem

The fix

How to set up and validate locally

The above videos explain how to reproduce the problem and how to verify the fix.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #376335 (closed)

Edited by Daniel Tian

Merge request reports