Skip to content

Avoid duplicate attribute search #3917

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

Closed
wants to merge 1 commit into from
Closed

Conversation

fhanik
Copy link
Contributor

@fhanik fhanik commented Jun 6, 2016

  • [] I have signed the CLA

When using search-and-bind strategy, the user attributes are already returned in the first search.
If the user happens to not have privileges to perform a search, the second search may fail.
(user only has bind privileges)
See cloudfoundry/uaa#342

@fhanik
Copy link
Contributor Author

fhanik commented Jun 6, 2016

UAA Tracker Story available at https://www.pivotaltracker.com/story/show/117634217

When using search-and-bind strategy, the user attributes are already returned in the first search.
If the user happens to not have privileges to perform a search, the second search may fail.
(user only has bind privileges)
See cloudfoundry/uaa#342
@pivotal-issuemaster
Copy link

pivotal-issuemaster commented Jun 9, 2016

@fhanik Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@rwinch rwinch self-assigned this Jun 17, 2016
@rwinch rwinch added in: ldap An issue in spring-security-ldap type: enhancement A general enhancement labels Jun 17, 2016
@rwinch
Copy link
Member

rwinch commented Jun 17, 2016

Thanks for the PR! This is now merged into master via 6b436ff

NOTE: I know that @fhanik has signed the old CLA so I am merging despite the status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: ldap An issue in spring-security-ldap type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants