LdapIgnoreTlsErrors and LdapIgnoreTlsValidation warnings are backwards

[JonathonReinhart]

As of version 4.2.1, these warnings appear to be backwards:

if (!_options.LdapIgnoreTlsErrors)
    _logger.LogWarning($"option [{nameof(_options.LdapIgnoreTlsErrors)}] is ENABLED; invalid certificates will be allowed");
else if (!_options.LdapIgnoreTlsValidation)
    _logger.LogWarning($"option [{nameof(_options.LdapIgnoreTlsValidation)}] is ENABLED; untrusted certificate roots will be allowed");

As far as I can tell, these options default to false, and the error message is printed even though I did not set the option:

warn: Zyborg.PassCore.PasswordProvider.LDAP.LdapPasswordChangeProvider[0]
      option [LdapIgnoreTlsErrors] is ENABLED; invalid certificates will be allowed

I think the solution is to remove the ! on both conditionals:

if (_options.LdapIgnoreTlsErrors)
    _logger.LogWarning($"option [{nameof(_options.LdapIgnoreTlsErrors)}] is ENABLED; invalid certificates will be allowed");
else if (_options.LdapIgnoreTlsValidation)
    _logger.LogWarning($"option [{nameof(_options.LdapIgnoreTlsValidation)}] is ENABLED; untrusted certificate roots will be allowed");

[JonathonReinhart]

Tangentially related to #263

[geoperez]

Today I'm going to verify your suggestion. Thanks!

[geoperez]

@JonathonReinhart can you take a look in my last commit?

[JonathonReinhart]

I haven't tried it, but that seems to make more sense. Thanks.