🐛 bug: fix client iterators when using break statement

client/request.go

@@ -298,11 +298,13 @@ func (r *Request) Cookie(key string) string {
 // Use maps.Collect() to gather them into a map if needed.
 func (r *Request) Cookies() iter.Seq2[string, string] {
 	return func(yield func(string, string) bool) {
-		r.cookies.VisitAll(func(key, val string) {
-			if !yield(key, val) {
+		var res bool
+		for k, v := range *r.cookies {
+			res = yield(k, v)
+			if !res {
 				return
 			}
-		})
+		}
 	}
 }
 
@@ -343,11 +345,11 @@ func (r *Request) PathParam(key string) string {
 // Use maps.Collect() to gather them into a map if needed.
 func (r *Request) PathParams() iter.Seq2[string, string] {
 	return func(yield func(string, string) bool) {
-		r.path.VisitAll(func(key, val string) {
-			if !yield(key, val) {
+		for k, v := range *r.path {
+			if !yield(k, v) {
 				return
 			}
-		})
+		}
 	}
 }
 

client/request_test.go

@@ -451,6 +451,14 @@
 	require.Equal(t, "bar", cookies["foo"])
 	require.Equal(t, "foo", cookies["bar"])
 
+	require.NotPanics(t, func() {
+		for _, v := range req.Cookies() {
+			if v == "bar" {
+				break
+			}
+		}
+	})
+
 	require.Len(t, cookies, 2)
 }
 
@@ -564,6 +572,14 @@
 	require.Equal(t, "foo", pathParams["bar"])
 
 	require.Len(t, pathParams, 2)
+
+	require.NotPanics(t, func() {
+		for _, v := range req.PathParams() {
+			if v == "bar" {
+				break
+			}
+		}
+	})
 }
 
 func Benchmark_Request_PathParams(b *testing.B) {
@@ -1579,7 +1595,7 @@
 
 		require.True(t, func() bool {
 			for _, v := range p.PeekMulti("TSlice") {
 				if string(v) == "bar" { //nolint:goconst // test
 					return true
 				}
 			}