Skip to content

Dont tag urls because performance

Cameron Swords requested to merge dont-tag-urls-because-performance into master

What does this MR do?

Removes tagging of URLs scanned because they take a long time to run, and don't appear to affect any of the execution of vulnerabilities.

An end-to-end test hasn't been added for this change, because the RegexAutoTagScanner ZAP class doesn't log, making it hard to verify what has taken place. Logging does occur when the scanner takes more than 5 seconds to scan a page, however, the effort to generate the page required for that is beyond the scope of this issue.

What are the relevant issue numbers?

gitlab-org/gitlab#230687 (closed)

Does this MR meet the acceptance criteria?

Edited by Cameron Swords

Merge request reports

Loading