fix: check HTTP status in ping pong clients

Author: markgoddard

workloads/ping-pong-cofide/ping-pong-cofide-client/main.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"io"
 	"log/slog"
+	"net/http"
 	"net/url"
 	"os"
 	"strconv"
@@ -127,6 +128,9 @@ func ping(client *cofidehttp.Client, serverAddr string, serverPort int) error {
 	if err != nil {
 		return err
 	}
+	if r.StatusCode != http.StatusOK {
+		return fmt.Errorf("unexpected status code: %d: %s", r.StatusCode, body[:1024])
+	}
 	slog.Info(string(body))
 	return nil
 }

workloads/ping-pong-mesh/ping-pong-mesh-client/main.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"context"
 	"fmt"
 	"io"
 	"log"
@@ -13,7 +12,7 @@ import (
 )
 
 func main() {
-	if err := run(context.Background(), getEnv()); err != nil {
+	if err := run(getEnv()); err != nil {
 		log.Fatal(err)
 	}
 }
@@ -38,7 +37,7 @@ func getEnv() *Env {
 	}
 }
 
-func run(ctx context.Context, env *Env) error {
+func run(env *Env) error {
 	client := &http.Client{
 		Transport: &http.Transport{},
 	}
@@ -69,6 +68,9 @@ func ping(client *http.Client, serverAddr, serverPort string) error {
 	if err != nil {
 		return err
 	}
+	if r.StatusCode != http.StatusOK {
+		return fmt.Errorf("unexpected status code: %d: %s", r.StatusCode, body[:1024])
+	}
 	slog.Info(string(body))
 	return nil
 }

workloads/ping-pong/ping-pong-client/main.go

@@ -217,6 +217,9 @@ func ping(client *http.Client, serverAddr string, serverPort int) error {
 	if err != nil {
 		return err
 	}
+	if r.StatusCode != http.StatusOK {
+		return fmt.Errorf("unexpected status code: %d: %s", r.StatusCode, body[:1024])
+	}
 	slog.Info(string(body))
 	return nil
 }