Only capture the first failure state in passport future listener (#1347)

Netflix · Oct 19, 2022 · 10f787a · 10f787a
1 parent a6cbfe9
commit 10f787a
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/zuul-core/src/main/java/com/netflix/zuul/netty/insights/PassportStateListener.java b/zuul-core/src/main/java/com/netflix/zuul/netty/insights/PassportStateListener.java
@@ -48,8 +48,12 @@ public void operationComplete(Future future) throws Exception
             passport.add(successState);
         }
         else {
-            if (failState != null)
-                passport.add(failState);
+            if (failState != null) {
+                // only capture a single failure state event,
+                // as sending content errors will fire for all content chunks,
+                // and we only need the first one
+                passport.addIfNotAlready(failState);
+            }
         }
     }
 }