Skip to content

Add support for custom LDAP parameter encoder #518

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

Add support for custom LDAP parameter encoder #518

wants to merge 5 commits into from

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
@mp911de
Polishing.
94b2168 
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
@marcingrzejszczak @mp911de
Preparing to work on the issue
b2c3717
@marcingrzejszczak @mp911de
Adds LdapParam and LdapEncoder
58b2d60 
fixes gh-509
@marcingrzejszczak @mp911de
Moved out of LdapParam to LdapEncode
2c2593e 
with this change we're explicitly adding an option to add an encoder on a concrete method parameter.
@mp911de
Polishing.
13f1095 
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
Add support for custom LDAP parameter encoder #509
Closed
@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
@marcingrzejszczak @mp911de
Add @LdapEncode annotation to configure encoding of LDAP parameters.
de3b8ff 
Closes #509
Original pull request: #518
mp911de added a commit that referenced this pull request Nov 18, 2024
@mp911de
Polishing.
c858f46 
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
@mp911de
Polishing.
877c583 
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
Reviewers
No reviews
Assignees

@mp911de mp911de

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
3.5 M1 (2025.0.0)
Development

Successfully merging this pull request may close these issues.

Add support for custom LDAP parameter encoder
2 participants
@mp911de @marcingrzejszczak