From a813b744c7c987e980e2f79384e0fcfd98b2136d Mon Sep 17 00:00:00 2001 From: labkey-tchad Date: Fri, 11 Jul 2025 15:29:26 -0700 Subject: [PATCH] Don't revisit actions after being redirected to them `.lastFilter` parameter can apply injection parameters when the crawler doesn't expect them --- src/org/labkey/test/util/Crawler.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/org/labkey/test/util/Crawler.java b/src/org/labkey/test/util/Crawler.java index 49963a2c74..fc3717aea6 100644 --- a/src/org/labkey/test/util/Crawler.java +++ b/src/org/labkey/test/util/Crawler.java @@ -997,6 +997,13 @@ private List crawlLink(final UrlToCheck urlToCheck) actualUrl = _test.getURL(); if (!actualUrl.toString().endsWith(relativeURL)) { + try + { + String actualRelativeUrl = WebTestHelper.makeRelativeUrl(actualUrl.toString()); + _actionsVisited.add(new ControllerActionId(actualRelativeUrl)); + _urlsVisited.add(actualRelativeUrl); + _urlsChecked.add(EscapeUtil.decodeUriPath(stripQueryParams(actualRelativeUrl))); + } catch (IllegalArgumentException ignored) {} originMessage = originMessage + "\nRedirected to: " + actualUrl; }