diff --git a/source/test-java/org/alfresco/repo/web/scripts/XssVulnerabilityTest.java b/source/test-java/org/alfresco/repo/web/scripts/XssVulnerabilityTest.java index 65b31a2696..98f66c76f9 100755 --- a/source/test-java/org/alfresco/repo/web/scripts/XssVulnerabilityTest.java +++ b/source/test-java/org/alfresco/repo/web/scripts/XssVulnerabilityTest.java @@ -27,7 +27,6 @@ import java.util.Set; import org.alfresco.error.AlfrescoRuntimeException; import org.alfresco.repo.security.authentication.AuthenticationUtil; -import org.alfresco.repo.web.scripts.tenant.TenantAdminSystemTest; import org.alfresco.rest.framework.core.exceptions.InvalidArgumentException; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -55,7 +54,7 @@ import org.springframework.extensions.webscripts.WebScriptException; */ public class XssVulnerabilityTest extends BaseWebScriptTest { - private static Log logger = LogFactory.getLog(TenantAdminSystemTest.class); + private Log logger = LogFactory.getLog(XssVulnerabilityTest.class); private DeclarativeRegistry webscriptsRegistry; @@ -92,6 +91,11 @@ public class XssVulnerabilityTest extends BaseWebScriptTest super.tearDown(); } + protected Log getLogger() + { + return logger; + } + public void testXssVulnerability() throws Throwable { webscriptsRegistry.reset(); @@ -100,9 +104,9 @@ public class XssVulnerabilityTest extends BaseWebScriptTest LinkedList vulnerabileURLS = new LinkedList(); for(WebScript ws : webscriptsRegistry.getWebScripts()) { - if (logger.isDebugEnabled()) + if (getLogger().isDebugEnabled()) { - logger.debug("progress: " + ++i + "/" + scriptsSize); + getLogger().debug("progress: " + ++i + "/" + scriptsSize); } Description wsDesc = ws.getDescription(); @@ -166,16 +170,16 @@ public class XssVulnerabilityTest extends BaseWebScriptTest } } - if (logger.isDebugEnabled()) + if (getLogger().isDebugEnabled()) { - logger.debug("OK html responses count: " + successCount); - logger.debug("Webscript errors count: " + wserrcount); - logger.debug("Vulnerabile URLs count: " + vulnCount); + getLogger().debug("OK html responses count: " + successCount); + getLogger().debug("Webscript errors count: " + wserrcount); + getLogger().debug("Vulnerabile URLs count: " + vulnCount); } for (String url : vulnerabileURLS) { - logger.warn("Vulnerabile URL: " + url); + getLogger().warn("Vulnerabile URL: " + url); } assertTrue("Vulnerabile URLs found: " + vulnerabileURLS, vulnerabileURLS.size() == 0); }