From 6c6114f0a108709fe191d8c282607e52f33cec3f Mon Sep 17 00:00:00 2001 From: gkolokythas Date: Thu, 14 Nov 2019 14:04:47 +0200 Subject: [PATCH] Makes "status" on Configurable Provider not mandatory and refactors "redirect_uri" to not include the extra "configurableId". --- .../configurable-login/configurable-login.component.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/dmp-frontend/src/app/ui/auth/login/configurable-login/configurable-login.component.ts b/dmp-frontend/src/app/ui/auth/login/configurable-login/configurable-login.component.ts index 278db1f85..dfeef2274 100644 --- a/dmp-frontend/src/app/ui/auth/login/configurable-login/configurable-login.component.ts +++ b/dmp-frontend/src/app/ui/auth/login/configurable-login/configurable-login.component.ts @@ -68,11 +68,12 @@ export class ConfigurableLoginComponent extends BaseComponent implements OnInit } public configurableAuthorize() { - window.location.href = this.provider.oauthUrl + let authUrl = this.provider.oauthUrl + '?response_type=code&client_id=' + this.provider.clientId - + '&redirect_uri=' + this.provider.redirect_uri + this.providerId - + '&state=' + this.provider.state + + '&redirect_uri=' + this.provider.redirect_uri + '&scope=' + this.provider.scope; + if (this.provider.state.length > 0) authUrl = authUrl + '&state=' + this.provider.state + window.location.href = authUrl; } public configurableLoginUser(code: string, state: string) {