Skip to content

Commit 6358dc3

Browse files
authored
ui: link account to LDAP - #13950 (#13950)
1 parent 7086030 commit 6358dc3

26 files changed

Lines changed: 154 additions & 9 deletions

File tree

‎plugins/user-authenticators/ldap/src/main/java/org/apache/cloudstack/api/command/LinkAccountToLdapCmd.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ public void execute() throws ServerApiException {
7979
if (admin != null) {
8080
LdapUser ldapUser = null;
8181
try {
82-
ldapUser = _ldapManager.getUser(admin, type, ldapDomain, domainId);
82+
ldapUser = _ldapManager.getUser(admin, getType(), ldapDomain, domainId);
8383
} catch (NoLdapUserMatchingQueryException e) {
8484
logger.debug("no ldap user matching username {} in the given group/ou", admin, e);
8585
}
@@ -120,7 +120,7 @@ public Long getDomainId() {
120120
}
121121

122122
public String getType() {
123-
return type;
123+
return type == null ? LdapManager.LinkType.GROUP.name() : type;
124124
}
125125

126126
public String getLdapDomain() {

‎plugins/user-authenticators/ldap/src/test/java/org/apache/cloudstack/api/command/LinkAccountToLdapCmdTest.java‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,4 +100,12 @@ public void execute() throws Exception {
100100
assertEquals("accountId", String.valueOf(accountId), result.getAdminId());
101101
assertEquals("accountName", String.valueOf(accountName), result.getAccountName());
102102
}
103+
104+
@Test
105+
public void getTypeDefaultsToGroup() throws Exception {
106+
assertEquals("GROUP", linkAccountToLdapCmd.getType());
107+
108+
setHiddenField(linkAccountToLdapCmd, "type", "OU");
109+
assertEquals("OU", linkAccountToLdapCmd.getType());
110+
}
103111
}

‎ui/public/locales/ar.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
"label.account.id": "Account ID",
1313
"label.account.name": "Account Name",
1414
"label.account.specific": "Account-Specific",
15+
"label.account.type.domain.admin": "مسؤول النطاق",
16+
"label.account.type.normal": "عادي (مستخدم)",
1517
"label.accounts": "Accounts",
1618
"label.accounttype": "Account Type",
1719
"label.acl.export": "Export ACL rules",
@@ -650,6 +652,7 @@
650652
"label.ldap.group.name": "LDAP Group",
651653
"label.level": "Level",
652654
"label.limitcpuuse": "CPU Cap",
655+
"label.link.account.to.ldap": "Link Account to LDAP",
653656
"label.link.domain.to.ldap": "Link Domain to LDAP",
654657
"label.linklocalip": "Control IP Address",
655658
"label.load.balancer": "Load Balancer",

‎ui/public/locales/ca.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
"label.account.id": "Account ID",
1313
"label.account.name": "Account Name",
1414
"label.account.specific": "Account-Specific",
15+
"label.account.type.domain.admin": "Administrador de domini",
16+
"label.account.type.normal": "Normal (Usuari)",
1517
"label.accounts": "Accounts",
1618
"label.accounttype": "Account Type",
1719
"label.acl.export": "Export ACL rules",
@@ -650,6 +652,7 @@
650652
"label.ldap.group.name": "LDAP Group",
651653
"label.level": "Level",
652654
"label.limitcpuuse": "CPU Cap",
655+
"label.link.account.to.ldap": "Link Account to LDAP",
653656
"label.link.domain.to.ldap": "Link Domain to LDAP",
654657
"label.linklocalip": "Control IP Address",
655658
"label.load.balancer": "Load Balancer",

‎ui/public/locales/de_DE.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
"label.account.id": "Benutzerkonto-ID",
2525
"label.account.name": "Benutzerkonto-Name",
2626
"label.account.specific": "Besonderheiten des Benutzerkontos",
27+
"label.account.type.domain.admin": "Domänen-Administrator",
28+
"label.account.type.normal": "Normal (Benutzer)",
2729
"label.accounts": "Benutzerkonten",
2830
"label.accounttype": "Benutzerkontotyp",
2931
"label.acl.export": "Export ACL rules",
@@ -836,6 +838,7 @@
836838
"label.limit": "Limit",
837839
"label.limitcpuuse": "CPU Obergrenze",
838840
"label.limits": "Limits konfigurieren",
841+
"label.link.account.to.ldap": "Verbinde Benutzerkonto mit LDAP",
839842
"label.link.domain.to.ldap": "Verbinde Domain mit LDAP",
840843
"label.linklocalip": "Link-Local IP-Adresse",
841844
"label.linux": "Linux",

‎ui/public/locales/el_GR.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@
3939
"label.account.id": "Αναγνωριστικό λογαριασμού",
4040
"label.account.name": "Όνομα λογαριασμού",
4141
"label.account.specific": "Ανά λογαριασμό",
42+
"label.account.type.domain.admin": "Διαχειριστής Τομέα",
43+
"label.account.type.normal": "Κανονικός (Χρήστης)",
4244
"label.accounts": "Λογαριασμοί",
4345
"label.accountstate": "Κατάσταση λογαριασμού",
4446
"label.accounttype": "Τύπος λογαριασμού",
@@ -1630,6 +1632,7 @@
16301632
"label.limits": "Όρια",
16311633
"label.limits.configure": "Ρύθμιση ορίων",
16321634
"label.link": "Σύνδεσμος",
1635+
"label.link.account.to.ldap": "Σύνδεση λογαριασμού με LDAP",
16331636
"label.link.domain.to.ldap": "Σύνδεση τομέα με LDAP",
16341637
"label.linklocalip": "Διεύθυνση IP Link-local/ελέγχου",
16351638
"label.linux": "Linux",

‎ui/public/locales/en.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@
4040
"label.account.id": "Account ID",
4141
"label.account.name": "Account name",
4242
"label.account.specific": "Account-specific",
43+
"label.account.type.domain.admin": "Domain Admin",
44+
"label.account.type.normal": "Normal (User)",
4345
"label.accounts": "Accounts",
4446
"label.accountstate": "Account state",
4547
"label.accounttype": "Account type",
@@ -1635,6 +1637,7 @@
16351637
"label.limits": "Limits",
16361638
"label.limits.configure": "Configure limits",
16371639
"label.link": "Link",
1640+
"label.link.account.to.ldap": "Link account to LDAP",
16381641
"label.link.domain.to.ldap": "Link domain to LDAP",
16391642
"label.linklocalip": "Link-local/Control IP address",
16401643
"label.linux": "Linux",

‎ui/public/locales/es.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
"label.account.id": "ID de la cuenta",
1313
"label.account.name": "Nombre de cuenta",
1414
"label.account.specific": "espec\u00edficas de la cuenta",
15+
"label.account.type.domain.admin": "Administrador de dominio",
16+
"label.account.type.normal": "Normal (Usuario)",
1517
"label.accounts": "Cuentas",
1618
"label.accounttype": "Tipo de Cuenta",
1719
"label.acl.export": "Export ACL rules",
@@ -667,6 +669,7 @@
667669
"label.ldap.group.name": "Grupo LDAP",
668670
"label.level": "Nivel",
669671
"label.limitcpuuse": "CPU Cap",
672+
"label.link.account.to.ldap": "Enlazar Cuenta a LDAP",
670673
"label.link.domain.to.ldap": "Enlazar Dominio a LDAP",
671674
"label.linklocalip": "Direcci\u00f3n IP de Enlace Local",
672675
"label.load.balancer": "Balanceador de carga",

‎ui/public/locales/fr_FR.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
"label.account.id": "ID de compte",
1313
"label.account.name": "Nom de compte",
1414
"label.account.specific": "Sp\u00e9cifique au compte",
15+
"label.account.type.domain.admin": "Administrateur de domaine",
16+
"label.account.type.normal": "Normal (Utilisateur)",
1517
"label.accounts": "Comptes",
1618
"label.accounttype": "Type Compte",
1719
"label.acl.export": "Export ACL rules",
@@ -650,6 +652,7 @@
650652
"label.ldap.group.name": "Groupe LDAP",
651653
"label.level": "Niveau",
652654
"label.limitcpuuse": "Limitation CPU",
655+
"label.link.account.to.ldap": "Lien Compte vers LDAP",
653656
"label.link.domain.to.ldap": "Lien Domaine vers LDAP",
654657
"label.linklocalip": "Adresse IP lien local",
655658
"label.load.balancer": "R\u00e9partiteur de charge",

‎ui/public/locales/hi.json‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
"label.access": "एक्सेस",
44
"label.accesskey": "एक्सेस की",
55
"label.account": "खाता",
6+
"label.account.type.domain.admin": "डोमेन व्यवस्थापक",
7+
"label.account.type.normal": "सामान्य (उपयोगकर्ता)",
68
"label.accounts": "लेखा",
79
"label.accounttype": "खाता प्रकार",
810
"label.aclid": "ACL",

0 commit comments

Comments
 (0)