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

[LDAP] searchDn and searchPassword should/could be optional #23087

Open
coudenysj opened this Issue Jun 7, 2017 · 5 comments

Comments

Projects
None yet
6 participants
@coudenysj

coudenysj commented Jun 7, 2017

Q A
Bug report? no
Feature request? yes
BC Break report? no
RFC? no
Symfony version 3.3.0 (but previous versions too)

I'm using the LDAP component to authenticate users in my application.

According to the docs I need a read only user to query the LDAP server, but I think this should/could be optional.

For the moment I customized the LdapUserProvider class, so the supplied username and password are used to authenticate (bind) to the LDAP server.

Is this a scenario that should be supported by the component? If so, I could work on a PR for this.

@fabpot

This comment has been minimized.

Show comment
Hide comment
@fabpot
Member

fabpot commented Jun 7, 2017

/cc @csarrazi

@csarrazi

This comment has been minimized.

Show comment
Hide comment
@csarrazi

csarrazi Jun 9, 2017

Contributor

Hi @coudenysj. No problem for this. However this will probably be merged in 3.4/master instead of 3.3, as the 3.3-beta has already been released at this point.

Contributor

csarrazi commented Jun 9, 2017

Hi @coudenysj. No problem for this. However this will probably be merged in 3.4/master instead of 3.3, as the 3.3-beta has already been released at this point.

@csarrazi

This comment has been minimized.

Show comment
Hide comment
@csarrazi

csarrazi Jun 9, 2017

Contributor

By the way, this is not related to the LDAP component, but actually to the security component (LdapUserProvider).

Contributor

csarrazi commented Jun 9, 2017

By the way, this is not related to the LDAP component, but actually to the security component (LdapUserProvider).

@chalasr

This comment has been minimized.

Show comment
Hide comment
@chalasr

chalasr Sep 3, 2017

Member

@coudenysj Do you still wish to submit a PR?

Member

chalasr commented Sep 3, 2017

@coudenysj Do you still wish to submit a PR?

@Simperfit

This comment has been minimized.

Show comment
Hide comment
@Simperfit

Simperfit Nov 1, 2017

Contributor

I can create a PR, what has to be done in here ?

Contributor

Simperfit commented Nov 1, 2017

I can create a PR, what has to be done in here ?

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