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.

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

Queries for objects with parentheses in the searched attribute value fail #5

Closed
Macmod opened this issue Apr 30, 2024 · 1 comment
Closed

Queries for objects with parentheses in the searched attribute value fail #5

Macmod opened this issue Apr 30, 2024 · 1 comment
Assignees
@Macmod
Labels
bug Something isn't working

Comments

@Macmod
Copy link
Owner

Macmod commented Apr 30, 2024

When an object which has a parenthesis in its searched attribute is queried the query fails, supposedly because some characters need to be escaped before appended to the LDAP query. This bug may exist in several functions and be present in several features.

I'm not sure atm whether there is a proper way to prepare/sanitize LDAP queries before appending clauses, but the solution should be in that direction.
@Macmod Macmod added the bug Something isn't working label Apr 30, 2024
@Macmod Macmod self-assigned this Apr 30, 2024
Macmod added a commit that referenced this issue Apr 30, 2024
@Macmod
Fixing issue #5 by escaping the fields of the main LDAP queries.
afc0a80
@Macmod
Copy link
Owner Author

Macmod commented Apr 30, 2024

Apparently ldap.EscapeFilter was all that was needed.

@Macmod Macmod closed this as completed Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Macmod Macmod

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Macmod