Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Filter raw values should not be escaped #6

Closed
jsumners opened this issue Aug 16, 2023 · 0 comments
Closed

Filter raw values should not be escaped #6

jsumners opened this issue Aug 16, 2023 · 0 comments

Comments

@jsumners
Copy link
Member

Every case of escapeFilterValue in the following should be removed:

value: escapeFilterValue(remainder)

The escapeFilterValue should only be used in the toString method of each filter. The raw value is the one sent to remote LDAP servers, and the specs dictate that the escape character should not be included. For example í should be sent as c3ad, not as \c3\ad.

This is the culprit behind ldapjs/node-ldapjs#860.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant