Skip to content
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

LDAPS - Misleading info result in thinking LDAPS is not working. #16847

Closed
2 tasks done
s0p4L1n3 opened this issue Mar 28, 2024 · 1 comment
Closed
2 tasks done

LDAPS - Misleading info result in thinking LDAPS is not working. #16847

s0p4L1n3 opened this issue Mar 28, 2024 · 1 comment

Comments

@s0p4L1n3
Copy link

Code of Conduct

  • I agree to follow this project's Code of Conduct

Is there an existing issue for this?

  • I have searched the existing issues

Version

10.0.14

Bug description

Based on my previous issue, which was closed too sooned by the bot.

I found the "problem" but as I can't make a PR, I propose these differents solution to helps the next adminsys (and avoid him to create another issue) that will try to configure LDAPS and thinking everything is correctly configured but no.

  • Not working LDAPS configuration
image
  • Working LDAPS configuration
image

Why do we have to specify the protocol ldaps:// in the server field when we are already setting port field with 636.
Then why we don't need to specify ldap:// when it is by default the port 389.

I propose different fixes:

  • Add a condition in the code that checks if the user configures port 636 but did not write ldaps:// in the server field, then a detailed error log informs the user.

  • Change the field description Server to something like this

image
  • Edit LDAP configuration interface:
    • replace the port field with an LDAP / LDAPS selection button
    • (depending on the choice, it will pre-fill the server field with ldaps:// and port 636 or the default ldap and 389)

Relevant log output

No response

Page URL

No response

Steps To reproduce

No response

Your GLPI setup information

No response

Anything else?

No response

Copy link
Contributor

This issue has been closed as we only track bugs here.

You can open a topic to discuss with community about this enhancement on suggestion website.
You can also contact GLPI editor team directly if you are willing to sponsor this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants