Skip to content

Commit f9b66c0

Browse files
zlamalpHejdaJakub
authored andcommitted
fix(profile): support adding both fed and cert identity types
DEPLOYMENT NOTE: Added new config option "consolidator_url_cert" in order to support different URLs for consolidator with federated and certificate authentication.
1 parent 316df47 commit f9b66c0

File tree

4 files changed

+10
-5
lines changed

4 files changed

+10
-5
lines changed

apps/user-profile/src/app/pages/identities-page/identities-page.component.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<div class="user-theme">
22
<h1 class="page-title">{{'IDENTITIES.IDP' | customTranslate | translate}}</h1>
3-
<button mat-flat-button color="accent" (click)="addIdentity()" [disabled]="loading">
3+
<button mat-flat-button color="accent" (click)="addIdentity(false)" [disabled]="loading">
44
{{'IDENTITIES.ADD' | customTranslate | translate}}
55
</button>
66
<button
@@ -23,7 +23,7 @@ <h1 class="page-title">{{'IDENTITIES.IDP' | customTranslate | translate}}</h1>
2323

2424
<div *ngIf="displayCertificates">
2525
<h1 class="page-title mt-5">{{'IDENTITIES.CERT' | customTranslate | translate}}</h1>
26-
<button mat-flat-button color="accent" (click)="addIdentity()" [disabled]="loading">
26+
<button mat-flat-button color="accent" (click)="addIdentity(true)" [disabled]="loading">
2727
{{'IDENTITIES.ADD' | customTranslate | translate}}
2828
</button>
2929
<button

apps/user-profile/src/app/pages/identities-page/identities-page.component.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ export class IdentitiesPageComponent implements OnInit {
125125
});
126126
}
127127

128-
addIdentity(): void {
128+
addIdentity(cert: boolean): void {
129129
if (this.storage.getProperty('use_new_consolidator')) {
130130
this.openLinkerService.openLinkerWindow((result: LinkerResult) => {
131131
if (result === 'TOKEN_EXPIRED') {
@@ -143,7 +143,10 @@ export class IdentitiesPageComponent implements OnInit {
143143
});
144144
} else {
145145
this.registrarManagerService.getConsolidatorToken().subscribe((token) => {
146-
const consolidatorUrl = this.storage.getProperty('consolidator_url');
146+
let consolidatorUrl = this.storage.getProperty('consolidator_url');
147+
if (cert) {
148+
consolidatorUrl = this.storage.getProperty('consolidator_url_cert');
149+
}
147150
window.location.href = `${consolidatorUrl}?target_url=${window.location.href}&token=${token}`;
148151
});
149152
}

apps/user-profile/src/assets/config/defaultConfig.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@
2828
"urn:perun:user:attribute-def:def:login-namespace:egi-ui",
2929
"urn:perun:user:attribute-def:def:login-namespace:sitola"
3030
],
31-
"consolidator_url": "https://perun-dev.cesnet.cz/cert-ic/ic/",
31+
"consolidator_url": "https://perun-dev.cesnet.cz/allfed-ic/ic/",
32+
"consolidator_url_cert": "https://perun-dev.cesnet.cz/cert-ic/ic/",
3233
"registrar_base_url": "https://perun-dev.cesnet.cz/fed/registrar/",
3334
"use_localhost_linker_url": false,
3435
"password_help": {

libs/perun/models/src/lib/ConfigProperties.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,7 @@ export interface PerunConfig {
187187
// Required
188188
displayed_tabs: string[];
189189
consolidator_url: string;
190+
consolidator_url_cert: string;
190191
registrar_base_url: string;
191192
mfa: ProfileMFA;
192193
preferred_unix_group_names: string[];

0 commit comments

Comments
 (0)