Update disable api calls logic

Author: DordeDimitrijev

src/script/acquisition-sdk.ts

@@ -58,14 +58,15 @@ export class AcquisitionStatus {
 }
 
 export class AcquisitionManager {
+    readonly BASER_URL = "https://codepush.appcenter.ms"
     private _appVersion: string;
     private _clientUniqueId: string;
     private _deploymentKey: string;
     private _httpRequester: Http.Requester;
     private _ignoreAppVersion: boolean;
     private _serverUrl: string;
     private _publicPrefixUrl: string = "v0.1/public/codepush/";
-    private _apiAttempts: number = 0;
+    private static _apiCallsDisabled: Boolean = false;
     constructor(httpRequester: Http.Requester, configuration: Configuration) {
         this._httpRequester = httpRequester;
 
@@ -80,8 +81,14 @@ export class AcquisitionManager {
         this._ignoreAppVersion = configuration.ignoreAppVersion;
     }
 
+    private disableApiCalls(statusCode: number) {
+        if (this._serverUrl.includes(this.BASER_URL) && !(statusCode >= 500 || statusCode == 408 || statusCode == 429)) {
+            AcquisitionManager._apiCallsDisabled = true
+        }
+    }
+
     public queryUpdateWithCurrentPackage(currentPackage: Package, callback?: Callback<RemotePackage | NativeUpdateNotification>): void {
-        if(this._apiAttempts>=1) return
+        if (AcquisitionManager._apiCallsDisabled) return
         if (!currentPackage || !currentPackage.appVersion) {
             throw new CodePushPackageError("Calling common acquisition SDK with incorrect package");  // Unexpected; indicates error in our implementation
         }
@@ -105,7 +112,7 @@ export class AcquisitionManager {
 
             if (response.statusCode !== 200) {
                 let errorMessage: any;
-                this._apiAttempts++
+                this.disableApiCalls(response.statusCode)
                 if (response.statusCode === 0) {
                     errorMessage = `Couldn't send request to ${requestUrl}, xhr.statusCode = 0 was returned. One of the possible reasons for that might be connection problems. Please, check your internet connection.`;
                 } else {
@@ -149,7 +156,7 @@ export class AcquisitionManager {
     }
 
     public reportStatusDeploy(deployedPackage?: Package, status?: string, previousLabelOrAppVersion?: string, previousDeploymentKey?: string, callback?: Callback<void>): void {
-        if(this._apiAttempts>=1) return
+        if (AcquisitionManager._apiCallsDisabled) return
         var url: string = this._serverUrl + this._publicPrefixUrl + "report_status/deploy";
         var body: DeploymentStatusReport = {
             app_version: this._appVersion,
@@ -200,7 +207,7 @@ export class AcquisitionManager {
                 }
 
                 if (response.statusCode !== 200) {
-                    this._apiAttempts++
+                    this.disableApiCalls(response.statusCode)
                     callback(new CodePushHttpError(response.statusCode + ": " + response.body), /*not used*/ null);
                     return;
                 }
@@ -211,7 +218,7 @@ export class AcquisitionManager {
     }
 
     public reportStatusDownload(downloadedPackage: Package, callback?: Callback<void>): void {
-        if(this._apiAttempts>=1) return
+        if (AcquisitionManager._apiCallsDisabled) return
         var url: string = this._serverUrl + this._publicPrefixUrl + "report_status/download";
         var body: DownloadReport = {
             client_unique_id: this._clientUniqueId,
@@ -227,7 +234,7 @@ export class AcquisitionManager {
                 }
 
                 if (response.statusCode !== 200) {
-                    this._apiAttempts++
+                    this.disableApiCalls(response.statusCode)
                     callback(new CodePushHttpError(response.statusCode + ": " + response.body), /*not used*/ null);
                     return;
                 }