[CORD-2827] Fixed unauthorized error handling
Change-Id: I6ddef7f869c17db4d8479f23f6e8734f6002d8fc
diff --git a/src/app/datasources/helpers/model-discoverer.service.ts b/src/app/datasources/helpers/model-discoverer.service.ts
index fca593c..3ee5a15 100644
--- a/src/app/datasources/helpers/model-discoverer.service.ts
+++ b/src/app/datasources/helpers/model-discoverer.service.ts
@@ -44,7 +44,7 @@
// Service
export interface IXosModelDiscovererService {
- discover(): ng.IPromise<boolean>;
+ discover(): ng.IPromise<string>;
getApiUrlFromModel(model: IXosModel): string;
areModelsLoaded(): boolean;
}
@@ -127,28 +127,42 @@
return this.$q.resolve('true');
})
.catch(err => {
- this.$log.error(`[XosModelDiscovererService] Model ${model.name} NOT stored`, err);
+ this.$log.warn(`[XosModelDiscovererService] Model ${model.name} NOT stored`, err);
+ const isAuthError = this.AuthService.isAuthError(err);
+ if (isAuthError) {
+ this.$log.warn(`[XosModelDiscovererService] User is not authentincated`);
+ return this.$q.reject(err);
+ }
return this.$q.resolve('false');
});
pArray.push(p);
});
+
+
this.$q.all(pArray)
.then((res) => {
- // the Model Loader promise won't ever be reject, in case it will be resolve with value false,
+ // the ModelLoader promise won't ever be reject, in case it will be resolve with value false,
// that's because we want to wait anyway for all the models to be loaded
if (res.indexOf('false') > -1) {
return d.resolve(false);
}
d.resolve(true);
+ this.modelsLoaded = true;
})
.catch((e) => {
- this.$log.error(`[XosModelDiscovererService]`, e);
- d.resolve(false);
+ this.XosModelStore.clean(); // reset all the observable otherwise they'll store login errors
+ this.$log.warn(`[XosModelDiscovererService]`, e);
+ // the ModelLoader promise will be rejected in case of authentication error
+ d.reject(e);
})
.finally(() => {
this.progressBar.complete();
- this.modelsLoaded = true;
});
+ })
+ .catch(err => {
+ this.progressBar.complete();
+ this.$log.error(`[XosModelDiscovererService] Cannot load model defs`, err);
+ return d.resolve('chameleon');
});
return d.promise;
}
@@ -249,16 +263,17 @@
}
private cacheModelEntries(model: IXosModel): ng.IPromise<IXosModel> {
+
const d = this.$q.defer();
const apiUrl = this.getApiUrlFromModel(model);
this.XosModelStore.query(model.name, apiUrl)
+ .skip(1) // NOTE observables returns as first an empty array, so skip it
.subscribe(
() => {
return d.resolve(model);
},
err => {
- this.AuthService.handleUnauthenticatedRequest(err);
return d.reject(err);
}
);