Update Retry logic to handle additional response codes (#1878)

microsoft · Aug 11, 2022 · ae6daa2 · ae6daa2
1 parent 6ba2c0d
commit ae6daa2
Show file tree

Hide file tree

Showing 2 changed files with 150 additions and 165 deletions.
diff --git a/channels/applicationinsights-channel-js/src/Sender.ts b/channels/applicationinsights-channel-js/src/Sender.ts
@@ -928,10 +928,14 @@ export class Sender extends BaseTelemetryPlugin implements IChannelControlsAI {
              * @param statusCode
              */
             function _isRetriable(statusCode: number): boolean {
-                return statusCode === 408 // Timeout
+                return statusCode === 401 // Unauthorized
+                    || statusCode === 403 // Forbidden
+                    || statusCode === 408 // Timeout
                     || statusCode === 429 // Too many requests.
                     || statusCode === 500 // Internal server error.
-                    || statusCode === 503; // Service unavailable.
+                    || statusCode === 502 // Bad Gateway.
+                    || statusCode === 503 // Service unavailable.
+                    || statusCode === 504; // Gateway timeout.
             }
 
             function _formatErrorMessageXhr(xhr: XMLHttpRequest, message?: string): string {