Retry API authentication errors since these can be transient

lib/analyze-action-post.js

@@ -160929,6 +160929,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/analyze-action.js

@@ -106639,6 +106639,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/autobuild-action.js

@@ -103300,6 +103300,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/init-action-post.js

@@ -164145,6 +164145,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/init-action.js

@@ -103986,6 +103986,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/resolve-environment-action.js

@@ -103308,6 +103308,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/setup-codeql-action.js

@@ -103417,6 +103417,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/start-proxy-action-post.js

@@ -160812,6 +160812,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/start-proxy-action.js

@@ -120388,6 +120388,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/upload-lib.js

@@ -106277,6 +106277,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/upload-sarif-action-post.js

@@ -160812,6 +160812,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

lib/upload-sarif-action.js

@@ -106323,6 +106323,12 @@ function createApiClientWithDetails(apiDetails, { allowExternal = false } = {})
         info: core5.info,
         warn: core5.warning,
         error: core5.error
+      },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451]
       }
     })
   );

src/api-client.test.ts

@@ -36,6 +36,9 @@ test("getApiClient", async (t) => {
       baseUrl: "http://api.github.localhost",
       log: sinon.match.any,
       userAgent: `CodeQL-Action/${actionsUtil.getActionVersion()}`,
+      retry: {
+        doNotRetry: [400, 410, 422, 451],
+      },
     }),
   );
 });

src/api-client.ts

@@ -51,6 +51,12 @@ function createApiClientWithDetails(
         warn: core.warning,
         error: core.error,
       },
+      retry: {
+        // The default is 400, 401, 403, 404, 410, 422, and 451. We have observed transient errors
+        // with authentication, so we remove 401, 403, and 404 from the default list to ensure that
+        // these errors are retried.
+        doNotRetry: [400, 410, 422, 451],
+      },
     }),
   );
 }