regex for ldapServer on storing in Manager is too string/wrong
The attached patch fixes it. Patch features:
- avoid leaning toothpick syndrome, use m{..} instead of //
- allow one letter hosts instead requiring 3 letters
- allow ldapi://, ldapi:/// and ldapi://var%2fldapi/
- when giving a port, require at least one digit
- forbid trailing slash on pure hosts [i.e. no URI]
- allow text after tailing slash on URIs [i.e. ldap+tls://host/tlsparam=x?tls2param=y works]
Please consider integrating it into the next version.
Best PETer