search

HealthCatalyst / Fabric.Identity

Identity service to provide authentication
Apache License 2.0
12 stars 6 forks source link

Change identity provider service used in AccountController away from idPSS #407

Closed amunar closed 4 years ago

amunar commented 4 years ago

This fixes an issue with Access Control login showing an error from Identity when UseDiscovery for Identity is enabled: https://dev.azure.com/healthcatalyst/CAP/_workitems/edit/196332

Replaced IExternalIdentityProviderService used during AccountController.ExternalLogin with IPrincipalSearchService. Registered IExternalIdentityProviderService class IdPSearchServiceProvider uses idPSS which may not be installed. Switching to IPrincipalSearchService (registered with PrincipalSearchService) uses code already inside Identity and moved from idPSS to try to search for the user.