Fixes a few problems in ClientUriSupplier implementations.

spericas · Sep 26, 2024 · a6c6d0f · a6c6d0f
1 parent ea12def
commit a6c6d0f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/webclient/grpc/src/main/java/io/helidon/webclient/grpc/ClientUriSuppliers.java b/webclient/grpc/src/main/java/io/helidon/webclient/grpc/ClientUriSuppliers.java
@@ -25,8 +25,8 @@
 import io.helidon.webclient.api.ClientUri;
 
 /**
- * Defines the {@code ClientUriSupplier} interface and provides a few basic
- * implementations of it.
+ * Some popular implementations of the {@link io.helidon.webclient.grpc.ClientUriSupplier}
+ * interface.
  */
 public class ClientUriSuppliers {
 
@@ -59,7 +59,7 @@ public static OrderedSupplier create(Collection<ClientUri> clientUris) {
         }
 
         protected OrderedSupplier(Collection<ClientUri> clientUris) {
-            this.clientUris = clientUris.iterator();
+            this.clientUris = List.copyOf(clientUris).iterator();
         }
 
         @Override
@@ -195,7 +195,7 @@ protected RandomSupplier(ClientUri[] clientUris) {
 
         @Override
         public boolean hasNext() {
-            return supplied;
+            return !supplied;
         }
 
         @Override