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

Add support for custom LDAP parameter encoder #518

Closed
wants to merge 5 commits into from
Closed

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Nov 18, 2024

Add @LdapEncode annotation to specify a custom LDAP parameter encoder.

mp911de and others added 5 commits November 18, 2024 09:06
Refactor ContextualValueExpressionEvaluator into Function, refactor BindingContext into nested class.

Simplify tests. Rename tests to reflect what they are actually doing. Reformat code.

See #453
Original pull request: #505
with this change we're explicitly adding an option to add an encoder on a concrete method parameter.
Refine Javadoc. Move LdapEncoder lookup into LdapParameters. Eagerly instantiate LdapEncoder. Refine method naming.
@mp911de mp911de self-assigned this Nov 18, 2024
@mp911de mp911de added the type: enhancement A general enhancement label Nov 18, 2024
@mp911de mp911de linked an issue Nov 18, 2024 that may be closed by this pull request
@mp911de mp911de added this to the 3.5 M1 (2025.0.0) milestone Nov 18, 2024
mp911de pushed a commit that referenced this pull request Nov 18, 2024
mp911de added a commit that referenced this pull request Nov 18, 2024
Refine Javadoc. Move LdapEncoder lookup into LdapParameters. Eagerly instantiate LdapEncoder. Refine method naming.

Add NameEncoder and LikeEncoder for simplified usage.

See #509
Original pull request: #518
@mp911de mp911de closed this Nov 18, 2024
@mp911de mp911de deleted the issue/509 branch November 18, 2024 08:32
mp911de added a commit that referenced this pull request Nov 18, 2024
Refine Javadoc. Move LdapEncoder lookup into LdapParameters. Eagerly instantiate LdapEncoder. Refine method naming.

Add NameEncoder and LikeEncoder for simplified usage.

See #509
Original pull request: #518
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for custom LDAP parameter encoder
2 participants