support for ldap servers requiring authentication for read access #259
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Have you read the docs first?
Yes
OK, describe you changes:
One line changer to cause Spring Security to trigger required config generation for LDAP Servers that require authentication.
Per Spring Docs, calling this method is required when implementations are outside of the Spring Context (which LDAPAuthenticator is): https://github.com/spring-projects/spring-ldap/blob/main/core/src/main/java/org/springframework/ldap/core/support/AbstractContextSource.java#L60-L62
Method call for reference: https://github.com/spring-projects/spring-ldap/blob/main/core/src/main/java/org/springframework/ldap/core/support/AbstractContextSource.java#L409
Tests?
Didn't add any since it's a one liner and would require a lot of setup for an in-memory secure ldap server