Api/UI: Update API docs or remove the name or ldapdomain parameter #11474
Description
problem
Update the API and UI implemenation for
https://cloudstack.apache.org/api/apidocs-4.20/apis/linkDomainToLdap.html
versions
ACS 4.20.1
The steps to reproduce the bug
Steps to reproduce the issue
- Execute the API
https://cloudstack.apache.org/api/apidocs-4.20/apis/linkDomainToLdap.html by providing the name and ldapdomain parameters
(localcloud) 🐱 > link domaintoldap domainid=394cbde8-efe2-4ef2-bac0-fa5958fa4134 type=GROUP accounttype=2 ldapdomain=cn=qa-team,dc=example,dc=in name=qa admin=admin
{
"LinkDomainToLdap": {
"accounttype": 2,
"domainid": "394cbde8-efe2-4ef2-bac0-fa5958fa4134",
"ldapdomain": "cn=qa-team,dc=example,dc=in",
"name": "cn=qa-team,dc=example,dc=in",
"type": "GROUP"
}
}
- Check the API esponse the name value (qa) is not stored anywhere and it's overwritten by the ldapdomain value
mysql> select * from ldap_trust_map;
+----+-----------+-------+-------------------------------------------+--------------+------------+
| id | domain_id | type | name | account_type | account_id |
+----+-----------+-------+-------------------------------------------+--------------+------------+
| 22 | 25 | GROUP | cn=dev-team,ou=Telco-Bng,dc=example,dc=in | 2 | 0 |
Also in the UI , we are using the name parameter
The description of the name should be updated,
What to do about it?
Either update the api docs to use one parameter
The Link domain to LDAP should say to give the full DN value
Reactions are currently unavailable