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

Bind via Kerberos #51

Open
luandrea opened this issue Nov 20, 2024 · 6 comments
Open

Bind via Kerberos #51

luandrea opened this issue Nov 20, 2024 · 6 comments

Comments

@luandrea
Copy link

As stated in the documentation:
For Kerberos authentication (GSSAPI) comment out comment out $ldap_bind*

I think it should be more correct to put them as null, otherwise you do not enter in the correct if in the bind class (Ldap.php).

I'm testing it with docker and I have to put it like this:
$ldap_binddn = null; $ldap_bindpw = null;

@luandrea
Copy link
Author

I'm referring to this documentation:
https://self-service-password.readthedocs.io/en/stable/config_ldap.html

It's about self-service-password application, but the source code is here.

@davidcoutadeur
Copy link

Yes, this is because $ldap_binddn and $ldap_bindpw are declared as default variables in config.inc.php:

$ldap_binddn = "cn=manager,dc=example,dc=com";
$ldap_bindpw = 'secret';

We should indeed update the doc to explain how to nullify these variables.

Thanks for the report.

@davidcoutadeur
Copy link

After verification, there is already a tip in ssp doc:

You can leave these parameters empty to bind anonymously or using GSSAPI (see below). In this case, the password modification must be done with user’s credentials. But this will not work for password reset.

I'll add more details.

davidcoutadeur pushed a commit to ltb-project/self-service-password that referenced this issue Nov 21, 2024
@davidcoutadeur
Copy link

@davidcoutadeur
Copy link

The fix is insufficient, as when:

$ldap_binddn = "";
$ldap_bindpw = "";

the authentication is still tried with ldap bind.

@luandrea
Copy link
Author

For me the correct setup was to put like this:

$ldap_binddn = null; 
$ldap_bindpw = null;

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

No branches or pull requests

2 participants