zavadil-ts-common 1.2.22 → 1.2.24

package/src/oauth/RestClientWithOAuth.ts

@@ -40,7 +40,8 @@ export class RestClientWithOAuth extends RestClient {
 			return this.setIdTokenRaw(urlToken)
 				.then(() => {
 					console.log("Redirecting to url without id token");
-					document.location = this.deleteIdTokenFromUrl();
+					this.deleteIdTokenFromUrl();
+					//document.location =;
 				});
 		} else {
 			const storageToken = this.getIdTokenFromLocalStorage();
@@ -86,10 +87,10 @@ export class RestClientWithOAuth extends RestClient {
 		return this.defaultPrivilege;
 	}
 
-	deleteIdTokenFromUrl(): string {
+	deleteIdTokenFromUrl() {
 		const up = new URLSearchParams(document.location.search);
 		up.delete(this.tokenName);
-		return up.toString();
+		document.location.search = up.toString();
 	}
 
 	getIdTokenFromUrl(): string | null {
@@ -169,19 +170,13 @@ export class RestClientWithOAuth extends RestClient {
 	}
 
 	/**
-	 * Try to obtain access token, then return true if everything is okay.
+	 * Try to obtain access token, then return if everything is okay.
 	 * This is basically only used when initializing and trying to determine whether we need to redirect user to login page
 	 */
-	checkAccessToken(privilege?: string): Promise<boolean> {
+	checkAccessToken(privilege?: string): Promise<any> {
 		return this.getTokenManager()
 			.then(tm => tm.getAccessToken(StringUtil.getNonEmpty(privilege, this.defaultPrivilege)))
-			.then((accessToken) => true)
-			.catch(
-				(e) => {
-					console.error('Access token check failed:', e);
-					return false;
-				}
-			);
+			.catch((e) => Promise.reject(`Access token check failed: ${e}`));
 	}
 
 }