interop client: fix soak test bug where we can crash if peer wasn't set

temawi · Jul 22, 2022 · d9d8a84 · d9d8a84
1 parent b193246
commit d9d8a84
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java b/interop-testing/src/main/java/io/grpc/testing/integration/AbstractInteropTest.java
@@ -2054,12 +2054,12 @@ public void performSoakTest(
             .withInterceptors(recordClientCallInterceptor(clientCallCapture));
       }
       SoakIterationResult result = performOneSoakIteration(soakStub);
-      String peer = clientCallCapture
-          .get().getAttributes().get(Grpc.TRANSPORT_ATTR_REMOTE_ADDR).toString();
+      SocketAddress peer = clientCallCapture
+          .get().getAttributes().get(Grpc.TRANSPORT_ATTR_REMOTE_ADDR);
       System.err.print(
           String.format(
               "soak iteration: %d elapsed_ms: %d peer: %s",
-              i, result.getLatencyMs(), peer));
+              i, result.getLatencyMs(), peer != null ? peer.toString() : "null"));
       if (!result.getStatus().equals(Status.OK)) {
         totalFailures++;
         System.err.println(String.format(" failed: %s", result.getStatus()));