Move HTTP client creation inside the API client retry callback block.

dart-lang · Feb 11, 2025 · da7469c · da7469c
1 parent e76fcac
commit da7469c
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/app/lib/tool/utils/pub_api_client.dart b/app/lib/tool/utils/pub_api_client.dart
@@ -116,17 +116,18 @@ Future<R> withRetryPubApiClient<R>(
   /// The enabled experiments that will be part of the experimental cookie.
   Set<String>? experiments,
 }) async {
-  final httpClient = httpClientWithAuthorization(
-    tokenProvider: () async => authToken,
-    sessionIdProvider: () async => sessionId,
-    csrfTokenProvider: () async => csrfToken,
-    cookieProvider: () async => {
-      if (experiments != null) experimentalCookieName: experiments.join(':'),
-    },
-    client: http.Client(),
-  );
   return await retry(
     () async {
+      final httpClient = httpClientWithAuthorization(
+        tokenProvider: () async => authToken,
+        sessionIdProvider: () async => sessionId,
+        csrfTokenProvider: () async => csrfToken,
+        cookieProvider: () async => {
+          if (experiments != null)
+            experimentalCookieName: experiments.join(':'),
+        },
+        client: http.Client(),
+      );
       try {
         final apiClient = PubApiClient(
           pubHostedUrl ?? activeConfiguration.primaryApiUri!.toString(),