Merge pull request #4510 from NativeScript/vladimirov/handle-http-304

fix: handle HTTP 304 response status code

Author: rosen-vladimirov

lib/common/constants.ts

@@ -92,6 +92,7 @@ export class Proxy {
  */
 export class HttpStatusCodes {
 	static SEE_OTHER = 303;
+	static NOT_MODIFIED = 304;
 	static PAYMENT_REQUIRED = 402;
 	static PROXY_AUTHENTICATION_REQUIRED = 407;
 }

lib/common/http-client.ts

@@ -168,7 +168,7 @@ export class HttpClient implements Server.IHttpClient {
 							this.setResponseResult(promiseActions, cleanupRequestData, { err: new Error(HttpClient.STUCK_RESPONSE_ERROR_MESSAGE) });
 						}
 					}, HttpClient.STUCK_RESPONSE_CHECK_INTERVAL);
-					const successful = helpers.isRequestSuccessful(responseData);
+					const successful = helpers.isRequestSuccessful(responseData) || responseData.statusCode === HttpStatusCodes.NOT_MODIFIED;
 					if (!successful) {
 						pipeTo = undefined;
 					}