Skip to content

Updating LDAP integration instructions #494

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

daviftorres
Copy link
Contributor

@daviftorres daviftorres commented Apr 11, 2025

Documentation page https://docs.cloudstack.apache.org/en/latest/adminguide/accounts.html
The instructions were transferred from bullet points to a table that organize in columns:


📚 Documentation preview 📚: https://cloudstack-documentation--494.org.readthedocs.build/en/494/

@DaanHoogland
Copy link
Contributor

thank you @daviftorres , I like your changes but there is a formatting issue with it.
image

It might be solved outside of the actual text changes, but I do not know how. Any idea?

@daviftorres
Copy link
Contributor Author

hey @DaanHoogland , it is because the following strings are single word and there is no word-wrapping:

  • OU=APAC,DC=company,DC=com
  • DC=company,DC=com
  • CN=ACSGroup,DC=company,DC=com
  • CN=ACSGroup,CN=Users,DC=company,DC=com
  • CN=ACSServiceAccount,OU=APAC,DC=company,DC=com
  • CN=ACSServiceAccount,CN=Users,DC=company,DC=com

Do you think if we introduce a "\n" it will mislead the users to think it is not a single-line string?

@daviftorres
Copy link
Contributor Author

@DaanHoogland I found out there is no flag for wrapping long words but I know that LDAP accepts spaces after the commas. So, I changed my deployment and added the spaces everywhere and CloudStack continue to work as before. Example:

  • DC=company, DC=com

@DaanHoogland
Copy link
Contributor

tnx @daviftorres

@DaanHoogland
Copy link
Contributor

btw @daviftorres , as this is based on main it will not reach the docs for the next dot-releases, only for v21. If you wish you can rebase it on 4.19 to have it in earlier.

@davift
Copy link

davift commented Apr 15, 2025

All good!
Thank YOU for your hard excellent work keeping this project.

@DaanHoogland DaanHoogland merged commit e8b7bf7 into apache:main Apr 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants