ASA 8404 (#218)

pom.xml

@@ -81,7 +81,7 @@
 	<dependency>
 		<groupId>com.hcl</groupId>
 		<artifactId>appscan.sdk</artifactId>
-		<version>1.1.2</version>
+		<version>1.1.3</version>
   	</dependency>
 	<dependency>
   	  	<groupId>org.apache.wink</groupId>

src/main/java/com/hcl/appscan/jenkins/plugin/actions/ScanResults.java

@@ -1,6 +1,6 @@
 /**
  * © Copyright IBM Corporation 2016.
- * @ Copyright HCL Technologies Ltd. 2019, 2020.
+ * @ Copyright HCL Technologies Ltd. 2019, 2024.
  * LICENSE: Apache License, Version 2.0 https://www.apache.org/licenses/LICENSE-2.0
  */
 
@@ -151,7 +151,7 @@ public File getReport() {
 	}
 
 	private String getReportName() {
-		String name = (getScanType() + FileUtil.getValidFilename(getName())).replaceAll(" ", ""); //$NON-NLS-1$ //$NON-NLS-2$
+		String name = FileUtil.getValidFilename(getName()).replaceAll(" ", ""); //$NON-NLS-1$ //$NON-NLS-2$
 		return name + REPORT_SUFFIX + "." + m_provider.getResultsFormat().toLowerCase(); //$NON-NLS-1$
 	}
 

src/main/java/com/hcl/appscan/jenkins/plugin/builders/AppScanBuildStep.java

@@ -305,27 +305,19 @@ private void shouldFailBuild(IResultsProvider provider,Run<?,?> build) throws Ab
 	    }
 	}
 
-    private void validations(boolean isAppScan360, Map<String, String> properties, IProgress progress, String target) throws AbortException {
+    private void validateGeneralSettings(boolean isAppScan360, Map<String, String> properties, IProgress progress) {
         if(isAppScan360) {
-            if (m_type.equals(Scanner.DYNAMIC_ANALYZER) && properties.containsKey(Scanner.PRESENCE_ID)) {
-                throw new AbortException(Messages.error_presence_AppScan360());
-            } if (m_type.equals(CoreConstants.SOFTWARE_COMPOSITION_ANALYZER)) {
-                throw new AbortException(Messages.error_sca_AppScan360());
-            } if (m_intervention) {
+            if (m_intervention) {
                 progress.setStatus(new Message(Message.WARNING, Messages.warning_allow_intervention_AppScan360()));
-            } if (properties.get(CoreConstants.OPEN_SOURCE_ONLY) != null) {
-                throw new AbortException(Messages.error_sca_AppScan360());
             }
         } else if (m_authProvider.getacceptInvalidCerts()) {
             progress.setStatus(new Message(Message.WARNING, Messages.warning_asoc_certificates()));
         }
 
-        if (m_type.equals(Scanner.STATIC_ANALYZER) && properties.containsKey(CoreConstants.OPEN_SOURCE_ONLY)) {
+        if(properties.containsKey(CoreConstants.OPEN_SOURCE_ONLY)) {
             progress.setStatus(new Message(Message.WARNING, Messages.warning_sca()));
-        }
-
-        if(!isAppScan360 && m_type.equals(Scanner.DYNAMIC_ANALYZER) && !properties.containsKey(Scanner.PRESENCE_ID) && !ServiceUtil.isValidUrl(target, m_authProvider, m_authProvider.getProxy())) {
-            throw new AbortException(Messages.error_url_validation(target));
+            m_scanner = ScannerFactory.getScanner(Scanner.SOFTWARE_COMPOSITION_ANALYZER, properties.get(CoreConstants.TARGET));
+            properties.put(CoreConstants.SCANNER_TYPE, CoreConstants.SOFTWARE_COMPOSITION_ANALYZER);
         }
     }
 
@@ -334,13 +326,15 @@ private void perform(Run<?,?> build, Launcher launcher, TaskListener listener) t
     	final IProgress progress = new ScanProgress(listener);
     	final boolean suspend = m_wait;
         Map<String, String> properties = getScanProperties(build,listener);
-        String target = properties.get(CoreConstants.TARGET);
-        final IScan scan = ScanFactory.createScan(properties, progress, m_authProvider);
         boolean isAppScan360 = ((JenkinsAuthenticationProvider) m_authProvider).isAppScan360();
 
-        validations(isAppScan360, properties, progress, target);
-
-    	IResultsProvider provider = launcher.getChannel().call(new Callable<IResultsProvider, AbortException>() {
+        m_scanner.validateSettings((JenkinsAuthenticationProvider) m_authProvider,properties, progress);
+        validateGeneralSettings(isAppScan360,properties,progress);
+
+
+        final IScan scan = ScanFactory.createScan(properties, progress, m_authProvider);
+
+        IResultsProvider provider = launcher.getChannel().call(new Callable<IResultsProvider, AbortException>() {
 			private static final long serialVersionUID = 1L;
 
 			@Override
@@ -353,7 +347,7 @@ public IResultsProvider call() throws AbortException {
 					setInstallDir();
 		    		scan.run();
 
-                                IResultsProvider provider=new NonCompliantIssuesResultProvider(scan.getScanId(), scan.getType(), scan.getServiceProvider(), progress);
+                                IResultsProvider provider=scan.getResultsProvider(true);
                                 provider.setReportFormat(scan.getReportFormat());
 		    		if(suspend) {
 		    			progress.setStatus(new Message(Message.INFO, Messages.analysis_running()));
@@ -387,8 +381,7 @@ public IResultsProvider call() throws AbortException {
 				  message += ", " + provider.getMessage();
 			  }
 			  build.setDescription(message);
-			  throw new AbortException(com.hcl.appscan.sdk.Messages.getMessage(ScanConstants.SCAN_FAILED, (" Scan Id: " + scan.getScanId() +
-					", Scan Name: " + scan.getName())));
+			  throw new AbortException(com.hcl.appscan.sdk.Messages.getMessage(ScanConstants.SCAN_FAILED, ("Scan Name: " + scan.getName())));
 		  }
         else if (CoreConstants.UNKNOWN.equalsIgnoreCase(m_scanStatus)) { // In case of internet disconnect Status is set to unstable.
             progress.setStatus(new Message(Message.ERROR, Messages.error_server_unavailable() + " "+ Messages.check_server(m_authProvider.getServer())));

src/main/java/com/hcl/appscan/jenkins/plugin/scanners/DynamicAnalyzer.java

@@ -12,6 +12,7 @@
 
 import com.hcl.appscan.jenkins.plugin.auth.ASoCCredentials;
 import com.hcl.appscan.jenkins.plugin.builders.AppScanBuildStep;
+import com.hcl.appscan.sdk.logging.IProgress;
 import org.jenkinsci.Symbol;
 import org.kohsuke.stapler.AncestorInPath;
 import org.kohsuke.stapler.DataBoundConstructor;
@@ -196,8 +197,17 @@ public String upgradeLoginScenario(){
 		}
 	}
 
+	public void validateSettings(JenkinsAuthenticationProvider authProvider, Map<String, String> properties, IProgress progress) throws AbortException {
+		if (authProvider.isAppScan360() && properties.containsKey(Scanner.PRESENCE_ID)) {
+			throw new AbortException(Messages.error_presence_AppScan360());
+		}
+        if (!authProvider.isAppScan360() && !properties.containsKey(Scanner.PRESENCE_ID) && !ServiceUtil.isValidUrl(properties.get(TARGET), authProvider, authProvider.getProxy())) {
+			throw new AbortException(Messages.error_url_validation(properties.get(TARGET)));
+		}
+	}
+
 	@Override
-	public Map<String, String> getProperties(VariableResolver<String> resolver) throws hudson.AbortException {
+	public Map<String, String> getProperties(VariableResolver<String> resolver) throws AbortException {
 		Map<String, String> properties = new HashMap<String, String>();
 		if (resolver == null) {
 			properties.put(TARGET, getTarget());
@@ -327,9 +337,9 @@ public FormValidation doCheckScanFile(@QueryParameter String scanFile) {
 
 		public FormValidation doCheckTarget(@QueryParameter String target,@RelativePath("..") @QueryParameter String credentials, @AncestorInPath ItemGroup<?> context, @QueryParameter String presenceId) {
 			JenkinsAuthenticationProvider authProvider = new JenkinsAuthenticationProvider(credentials,context);
-            		if(presenceId != null && presenceId.equals(EMPTY) && !target.equals(EMPTY) && !ServiceUtil.isValidUrl(target, authProvider, authProvider.getProxy())) {
-                		return FormValidation.error(Messages.error_url_validation_ui());
-            		}
+			if(!authProvider.isAppScan360() && presenceId != null && presenceId.equals(EMPTY) && !target.equals(EMPTY) && !ServiceUtil.isValidUrl(target, authProvider, authProvider.getProxy())) {
+				return FormValidation.error(Messages.error_url_validation_ui());
+			}
 			return FormValidation.validateRequired(target);
 		}
 

src/main/java/com/hcl/appscan/jenkins/plugin/scanners/Scanner.java

@@ -1,11 +1,13 @@
 /**
  * @ Copyright IBM Corporation 2016.
- * @ Copyright HCL Technologies Ltd. 2017, 2019.
+ * @ Copyright HCL Technologies Ltd. 2017, 2024.
  * LICENSE: Apache License, Version 2.0 https://www.apache.org/licenses/LICENSE-2.0
  */
 
 package com.hcl.appscan.jenkins.plugin.scanners;
 
+import com.hcl.appscan.jenkins.plugin.auth.JenkinsAuthenticationProvider;
+import com.hcl.appscan.sdk.logging.IProgress;
 import hudson.AbortException;
 import hudson.Util;
 import hudson.model.AbstractDescribableImpl;
@@ -38,7 +40,9 @@ public String getTarget() {
 	}
 
 	public abstract Map<String, String> getProperties(VariableResolver<String> resolver) throws AbortException;
-
+
+	public abstract void validateSettings(JenkinsAuthenticationProvider authProvider, Map<String, String> properties, IProgress progress) throws AbortException;
+
 	public abstract String getType();
 
 	protected String resolvePath(String path, VariableResolver<String> resolver) {

src/main/java/com/hcl/appscan/jenkins/plugin/scanners/SoftwareCompositionAnalyzer.java

@@ -1,12 +1,14 @@
 /**
- * @ Copyright HCL Technologies Ltd. 2023.
+ * @ Copyright HCL Technologies Ltd. 2023, 2024.
  * LICENSE: Apache License, Version 2.0 https://www.apache.org/licenses/LICENSE-2.0
  */
 
 package com.hcl.appscan.jenkins.plugin.scanners;
 
 import com.hcl.appscan.jenkins.plugin.Messages;
 import com.hcl.appscan.jenkins.plugin.auth.JenkinsAuthenticationProvider;
+import com.hcl.appscan.sdk.logging.IProgress;
+import hudson.AbortException;
 import hudson.Extension;
 import hudson.RelativePath;
 import hudson.model.ItemGroup;
@@ -37,7 +39,13 @@ public String getType() {
         return SOFTWARE_COMPOSITION_ANALYZER;
     }
 
-    public Map<String, String> getProperties(VariableResolver<String> resolver) {
+    public void validateSettings(JenkinsAuthenticationProvider authProvider, Map<String, String> properties, IProgress progress) throws AbortException {
+        if (authProvider.isAppScan360()) {
+            throw new AbortException(Messages.error_sca_AppScan360());
+        }
+    }
+
+    public Map<String, String> getProperties(VariableResolver<String> resolver) throws AbortException {
         Map<String, String> properties = new HashMap<String, String>();
         properties.put(TARGET, resolver == null ? getTarget() : resolvePath(getTarget(), resolver));
         return properties;