ag-admin/OpenSlides
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Allow reload on login children

Browse Source 
Fixes a bug where the user was always redirected to
/login/ as unknown user, when they tried to
load children from login page from an external references.
This commit is contained in:
Sean Engelhardt 2019-02-04 15:17:03 +01:00
parent 27ecb6f411
commit 06bfa314cb
2 changed files with 28 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
client/src/app/core/core-services/openslides.service.ts
View File

@ -58,8 +58,17 @@ export class OpenSlidesService extends OpenSlidesComponent {
this.operator.guestsEnabled = response.guest_enabled; this.operator.guestsEnabled = response.guest_enabled;
if (!response.user && !response.guest_enabled) { if (!response.user && !response.guest_enabled) {
this.redirectUrl = location.pathname; this.redirectUrl = location.pathname;
// Goto login, if the user isn't login and guests are not allowed
this.router.navigate(['/login']); // let the use navigate and reload on every login-page
if (this.redirectUrl.includes('/login/')) {
// Allow free navigation in the children of the login page
// required for resetting password and direct navigation to legal notice
// and privacy policy.
this.router.navigate([this.redirectUrl]);
} else {
// Goto login, if the user isn't login and guests are not allowed
this.router.navigate(['/login']);
}
} else { } else {
await this.afterLoginBootup(response.user_id); await this.afterLoginBootup(response.user_id);
} }

32
client/src/app/site/login/components/reset-password-confirm/reset-password-confirm.component.ts
View File

@ -58,21 +58,23 @@ export class ResetPasswordConfirmComponent extends BaseComponent implements OnIn
public ngOnInit(): void { public ngOnInit(): void {
super.setTitle('Reset password'); super.setTitle('Reset password');
this.activatedRoute.queryParams.subscribe(params => { this.activatedRoute.queryParams.subscribe(params => {
if (!params.user_id || !params.token) { if (!this.user_id && !this.token) {
setTimeout(() => { if (!params.user_id || !params.token) {
this.matSnackBar.open(''); setTimeout(() => {
this.matSnackBar.open( this.matSnackBar.open('');
this.translate.instant('The link is broken. Please contact your system administrator.'), this.matSnackBar.open(
this.translate.instant('OK'), this.translate.instant('The link is broken. Please contact your system administrator.'),
{ this.translate.instant('OK'),
duration: 0 {
} duration: 0
); }
this.router.navigate(['/']); );
}); this.router.navigate(['/login']);
} else { });
this.user_id = params.user_id; } else {
this.token = params.token; this.user_id = params.user_id;
this.token = params.token;
}
} }
}); });
} }