Merge pull request #9509 from abpframework/auto-merge/rel-4-4/467

Merge branch dev with rel-4.4
pull/9511/head
maliming 4 years ago committed by GitHub
commit 3838ef7b4a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -12,11 +12,11 @@ namespace Volo.Abp.Ldap
public class LdapManager : ILdapManager, ITransientDependency
{
public ILogger<LdapManager> Logger { get; set; }
protected AbpLdapOptions LdapOptions { get; }
protected IOptions<AbpLdapOptions> LdapOptions { get; }
public LdapManager(IOptions<AbpLdapOptions> ldapSettingsOptions)
public LdapManager(IOptions<AbpLdapOptions> ldapOptions)
{
LdapOptions = ldapSettingsOptions.Value;
LdapOptions = ldapOptions;
Logger = NullLogger<LdapManager>.Instance;
}
@ -50,11 +50,11 @@ namespace Volo.Abp.Ldap
return Task.CompletedTask;
}
protected virtual Task ConnectAsync(ILdapConnection ldapConnection)
protected virtual async Task ConnectAsync(ILdapConnection ldapConnection)
{
ldapConnection.Connect(LdapOptions.ServerHost, LdapOptions.ServerPort);
await LdapOptions.SetAsync();
return Task.CompletedTask;
ldapConnection.Connect(LdapOptions.Value.ServerHost, LdapOptions.Value.ServerPort);
}
protected virtual async Task AuthenticateLdapConnectionAsync(ILdapConnection connection, string username, string password)

Loading…
Cancel
Save