PD-5437 (#2814)

Author: cryptalith

src/app/core/http/retry-transient.interceptor.ts

@@ -9,7 +9,7 @@ import { Injectable } from '@angular/core'
 import { Observable, timer } from 'rxjs'
 import { retry } from 'rxjs/operators'
 
-const RETRYABLE_HTTP_STATUSES = new Set([408, 502, 503, 504])
+const RETRYABLE_HTTP_STATUSES = new Set([0, 408, 502, 503, 504])
 const MAX_RETRIES = 2
 
 @Injectable()

src/app/core/http/retry-transient.spec.ts

@@ -3,6 +3,32 @@ import { defer, of, throwError } from 'rxjs'
 import { retryTransient } from './retry-transient'
 
 describe('retryTransient', () => {
+  it('retries up to the max retries for HTTP status 0', (done) => {
+    let attempts = 0
+    const source$ = defer(() => {
+      attempts++
+      if (attempts <= 2) {
+        return throwError(
+          () =>
+            new HttpErrorResponse({
+              status: 0,
+              statusText: 'Network error',
+            })
+        )
+      }
+      return of('ok')
+    })
+
+    source$.pipe(retryTransient(2)).subscribe({
+      next: (value) => {
+        expect(value).toBe('ok')
+        expect(attempts).toBe(3)
+        done()
+      },
+      error: done.fail,
+    })
+  })
+
   it('retries up to the max retries for retryable HTTP statuses', (done) => {
     let attempts = 0
     const source$ = defer(() => {

src/app/core/http/retry-transient.ts

@@ -2,7 +2,7 @@ import { HttpErrorResponse } from '@angular/common/http'
 import { MonoTypeOperatorFunction, timer } from 'rxjs'
 import { retry } from 'rxjs/operators'
 
-export const RETRYABLE_HTTP_STATUSES = new Set([408, 502, 503, 504])
+export const RETRYABLE_HTTP_STATUSES = new Set([0, 408, 502, 503, 504])
 
 export function retryTransient<T>(
   maxRetries = 2,