The change_password method is broken ! about exldap HOT 4 OPEN

I primarily use this library against Active Directory and unfortunately it doesn't follow RFC 3062.

We could add in modify_password/3 and modify_password/4, we would just have to explain in the docs why there is change_password and modify_password and when to use one or the other.

from exldap.

Having both modify_password and change_password is seriously confusing. The ideal would be to add an ad_ prefix/suffix/parameter [0] to the original method, but will break compatibility with older releases...

[0] I think the AD case should be the "special case" since it doesn't follow the RFC.

from exldap.

Or some kind of server_type/is_active_directory parameter defaulting to the "AD way" to avoid breaking existing code, while keeping a somewhat sane API ?

from exldap.

Yeah I agree that it would be pretty confusing having both modify_password and change_password.
I like the server_type/is_active_directory idea.

server_type: :active_directory | :ldap

from exldap.