Merge pull request #8 from ShiftLeftSecurity/max/saferDependencyNodes

Safer NewDependency node creation
ShiftLeftSecurity · Sep 1, 2021 · f9755fb · f9755fb
2 parents 02c8874 + fc4fc0d
commit f9755fb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/scala/io/shiftleft/js2cpg/cpg/passes/astcreation/AstNodeBuilder.scala b/src/main/scala/io/shiftleft/js2cpg/cpg/passes/astcreation/AstNodeBuilder.scala
@@ -55,9 +55,9 @@ class AstNodeBuilder[NodeBuilderType](private val diffGraph: DiffGraph.Builder,
 
   def createDependencyNode(name: String, groupId: String, version: String): NewDependency = {
     val dependency = NewDependency()
-      .version(version)
-      .name(name)
-      .dependencyGroupId(Some(groupId))
+      .name(Option(name).getOrElse("<n/a>"))
+      .dependencyGroupId(Option(groupId).getOrElse("<n/a>"))
+      .version(Option(version).getOrElse("<n/a>"))
       .build
     diffGraph.addNode(dependency)
     dependency