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

case insensitive routing #20

Closed
nrogers opened this issue Sep 29, 2011 · 3 comments
Closed

case insensitive routing #20

nrogers opened this issue Sep 29, 2011 · 3 comments

Comments

@nrogers
Copy link
Contributor

nrogers commented Sep 29, 2011

Currently if I create a route for ou=base, the router won't send it requests for OU=base, ou=Base, etc. I'd like some way to create a route that ignores case when comparing the request and route DNs.

I think most LDAP servers generally do case insensitive comparisons. It might make sense to make that the default in ldapjs. There could be a server-level 'case sensitive routes' option like in express to override that if needed.

@mcavage
Copy link
Contributor

mcavage commented Sep 29, 2011

Hi,

So attribute names (ou) being case-sensitive is just a bug. That I'll take care of. Usually though LDAP is case-sensitive with the values (including in a DN), but iirc AD is case insensitive.

Do you need both? Or just attributes? (attributes I'll definitely fix, and does not need an option).

@nrogers
Copy link
Contributor Author

nrogers commented Sep 29, 2011

OK, that makes sense. The attribute names are the main thing I was worried about.

It might be nice to have an option for the values, but it's not a high priority for me.

@mcavage
Copy link
Contributor

mcavage commented Sep 29, 2011

This will be in the next npm version (0.1.9). I tend to wait a few days to batch these up, so if you need it urgently, let me know.

@mcavage mcavage closed this as completed Sep 29, 2011
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

2 participants