fix: Use orElseGet to defer loading of default credentials. (#124)

mrdziuban · web-flow · commit 6adac0effbd5 · 2025-04-03T09:36:39.000-07:00
diff --git a/src/main/java/com/google/genai/ApiClient.java b/src/main/java/com/google/genai/ApiClient.java
@@ -104,7 +104,7 @@ abstract class ApiClient {
       throw new IllegalArgumentException("Location must not be empty.");
     }
 
-    this.credentials = Optional.of(credentials.orElse(defaultCredentials()));
+    this.credentials = Optional.of(credentials.orElseGet(() -> defaultCredentials()));
 
     this.httpOptions = defaultHttpOptions(/* vertexAI= */ true, this.location);
 

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ abstract class ApiClient {`
`104`	`104`	`throw new IllegalArgumentException("Location must not be empty.");`
`105`	`105`	`}`
`106`	`106`
`107`		`- this.credentials = Optional.of(credentials.orElse(defaultCredentials()));`
	`107`	`+ this.credentials = Optional.of(credentials.orElseGet(() -> defaultCredentials()));`
`108`	`108`
`109`	`109`	`this.httpOptions = defaultHttpOptions(/* vertexAI= */ true, this.location);`
`110`	`110`