ldap: Properly escape search and dn values for queries

[greezybacon]

Fixes osTicket/osTicket#2582
Fixes osTicket/osTicket#2336
Fixes osTicket/osTicket#1422

[RauPe2]

Hi greezybacon,
i tryed your patch, but it doesnt work.
With your changes, the ldap-search returns no results from the Active Directory.
no remote-users - only local-users!

i can not insert a remote-user from the pulldown-menu into the osticket user-list.

We have users e.g. 'CN=lastname, firstname'
Users e.g. 'CN= lastname firstname' work
do you have any idea?

[jdelhome3578]

@greezybacon Can this be merged and a new phar file created?

[Micke1101]

Hi,
I wanted to check if anyone got this working as i've tried to make the phar myself without result.
I'm able to login as staff and client (including registration) and also when i add staff members the search works but when trying to add normal users i don't get any ldap suggestions, only the ones that are already added if i enter the full email.
Osticket: 1.10 RC3
PHP: 5.6.19
Apache: 2.4.18
Mysql: 5.7.11
Ldap type: Microsoft active directory
OS: Microsoft server 2008
Browsers: Chrome and Internet Explorer

[Neustradamus]

@greezybacon, @RauPe2, @jdelhome3578, @Micke1101: What is the status of this PR several years after?