Updated collection process to return 'No Certificate Collected' outco…

…me when a certificate isn't present in the configured header
FrankGasparovic · Dec 2, 2021 · c84ec94 · c84ec94
1 parent 74a15a7
commit c84ec94
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -13,7 +13,7 @@
  * information: "Portions copyright [year] [name of copyright owner]".
  *
  * Copyright 2017-2018 ForgeRock AS.
- * 
+ *
 -->
 
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
@@ -24,7 +24,7 @@
     <artifactId>certificate-node</artifactId>
 
     <groupId>org.forgerock.am</groupId>
-    <version>1.0.0</version>
+    <version>1.0.1</version>
     <name>Template for an Auth Tree Node maven project</name>
     <description>An Authentication Tree Node for ForgeRock's Identity Platform</description>
 

diff --git a/src/main/java/org/forgerock/openam/auth/nodes/CertificateCollectorNode.java b/src/main/java/org/forgerock/openam/auth/nodes/CertificateCollectorNode.java
@@ -156,7 +156,8 @@ private X509Certificate[] getPortalStyleCert(ListMultimap<String, String> header
         }
         logger.debug("Validate cert: " + cert);
         if (cert == null || cert.equals("")) {
-            throw new NodeProcessException("Certificate: no cert from HttpServletRequest header");
+            logger.debug("Certificate: no cert from HttpServletRequest header");
+            return null;
         }
 
         byte[] decoded = Base64.decode(cert);