Skip to content

Conversation

@labkey-tchad
Copy link
Member

Rationale

A previous change made Crawler.ControllerActionId decode the containerPath when parsing a URL. I missed a place that was assuming otherwise.

java.lang.IllegalArgumentException: URLDecoder: Illegal hex characters in escape (%) pattern - Error at index 0 in: "%+"
  at java.base/java.net.URLDecoder.decode(URLDecoder.java:237)
  at org.labkey.test.util.EscapeUtil.decode(EscapeUtil.java:81)
  at org.labkey.test.util.Crawler$UrlToCheck.underCreatedProject(Crawler.java:594)
  at org.labkey.test.util.Crawler$UrlToCheck.<init>(Crawler.java:521)

Related Pull Requests

Changes

  • Don't decode the result of ControllerActionId.getFolder

@labkey-tchad labkey-tchad changed the title Don't decode the result of ControllerActionId.getFolder Don't decode the result of ControllerActionId.getContainerPath Nov 21, 2024
@labkey-tchad labkey-tchad merged commit 6454021 into develop Nov 22, 2024
@labkey-tchad labkey-tchad deleted the fb_crawlerFolderEncoding branch November 22, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants