Prevent StackOverflowError in FlowNode#getParents()

Co-authored-by: Victor Martinez <[email protected]>

Author: cyrille-leclerc

src/main/java/io/jenkins/plugins/opentelemetry/job/OtelTraceService.java

@@ -31,7 +31,11 @@
 import javax.annotation.Nonnull;
 import javax.annotation.concurrent.Immutable;
 import javax.inject.Inject;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.List;
+import java.util.Objects;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
 import java.util.logging.Level;
@@ -147,7 +151,13 @@ public void buildListOfAncestors(@Nonnull FlowNode flowNode, @Nonnull List<FlowN
             flowNode = startNode;
         }
         for (FlowNode parentNode : flowNode.getParents()) {
-            buildListOfAncestors(parentNode, parents);
+            if (flowNode.equals(parentNode)) {
+                LOGGER.log(Level.INFO, "buildListOfAncestors(" + flowNode + "): skip parentFlowNode as it is the current node"); // TODO change message to Level.FINE once the cause is understood
+            } else if (parents.contains(parentNode)) {
+                LOGGER.log(Level.INFO, "buildListOfAncestors(" + flowNode + "): skip already added " + parentNode);  // TODO can we remove this check once the cause is understood?
+            } else {
+                buildListOfAncestors(parentNode, parents);
+            }
         }
     }