diff --git a/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/AcToolUiService.java b/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/AcToolUiService.java index e6b5084c..b79988a1 100644 --- a/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/AcToolUiService.java +++ b/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/AcToolUiService.java @@ -576,8 +576,9 @@ private void printInstallationLogsSection(HtmlWriter writer, RequestParameters r return; } + String sanitizedCurrentPath = escapeHtml4(currentPath); for (AcToolExecution acToolExecution : acToolExecutions.values()) { - String linkToLog = currentPath + "?" + PARAM_SHOW_LOG_ID + "=" + acToolExecution.getId(); + String linkToLog = sanitizedCurrentPath + "?" + PARAM_SHOW_LOG_ID + "=" + acToolExecution.getId(); String downloadLinkToLog = basePath + "/" + SUFFIX_DOWNLOAD_LOG + "?" + PARAM_SHOW_LOG_ID + "=" + acToolExecution.getId(); writer.tr(); writer.openTd(); @@ -610,7 +611,7 @@ private void printInstallationLogsSection(HtmlWriter writer, RequestParameters r String logHtml = acHistoryService.getLogFromHistory(reqParams.showLogId, true, reqParams.showLogVerbose, MAX_LINE_WIDTH); writer.openTable("logTable"); - writer.tableHeader(logLabel, 1, false); + writer.tableHeader(logLabel, 1); writer.tr(); writer.openTd(); writer.println(logHtml); diff --git a/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/HtmlWriter.java b/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/HtmlWriter.java index 7df617ef..1546c724 100644 --- a/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/HtmlWriter.java +++ b/accesscontroltool-bundle/src/main/java/biz/netcentric/cq/tools/actool/ui/HtmlWriter.java @@ -37,14 +37,6 @@ void closeTable() { pw.println(""); } - void tableHeader(String title, int colspan, boolean escape) { - tr(); - pw.print("