Case sensitivity for logins about redmine_ldap_sync HOT 5 CLOSED

What's your database?

• MySQL is case insensitive by default.

select * from users where login = 'SomeUser';
and select * from users where login = 'someuser';

... should return the same user.

• If you're using PostgreSQL it's a bit more complicated:

You can try using the plugin: http://www.postgresql.org/docs/9.1/static/citext.html

 CREATE EXTENSION citext;
 ALTER TABLE users ALTER COLUMN login TYPE citext;

To support PostgreSQL I could provide a configurable option to ensure that all users are created and searched with a lowercase login.
But on login both SomeUser and someuser will still be considered as different users.

from redmine_ldap_sync.

Well, we use MySQL, and as a software developer I can say the query should not be case-insensitive by default, unless specifically said so (usually by WHERE lower(login) = lower('SomeUser') or like that).

I think, the comparison should be made inside LDAP Sync script only, thus not altering the logic of Redmine as a whole (we have three working auth sources — local, ldap, kerberos — and I'm not sure how this trick will afffect each of them).

Am I wrong?

from redmine_ldap_sync.

It actually can depend (for example in Oracle you can specify a collation for the database, the session, on the query to make the comparisons case insensitive. Eg. Latin1_General_CI_AI).

I understand what you're telling. I've tried the query on my MySQL server and it returns the same.
What I've told you was based both on the test and some searchs:
http://stackoverflow.com/questions/5629111/how-can-i-make-sql-case-sensitive-string-comparison-on-mysql

LDAP Sync can only do the same that redmine is doing. If redmine considers SomeUser and someuser as different users the plugin will have to do the same.

I'll do some tests on my enviromnent and see what can be done.

from redmine_ldap_sync.

It's a bug. Redmine considers both SomeUser and someuser as the same.
It will be fixed when possible.

from redmine_ldap_sync.

This should be fixed now.

Feel free to reopen the ticket if you need further help.

from redmine_ldap_sync.