Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional authentication options #1289

Open
DavidBoike opened this issue Nov 29, 2022 · 2 comments
Open

Additional authentication options #1289

DavidBoike opened this issue Nov 29, 2022 · 2 comments
Labels

Comments

@DavidBoike
Copy link
Member

DavidBoike commented Nov 29, 2022

Basic authentication

If installing ServicePulse in IIS and securing ServiceControl APIs, Windows authentication must be used or ServiceInsight won't be able to connect to the necessary APIs, as shown in the limitations section:

If ServiceControl is secured with an authentication module other that Windows authentication, ServiceInsight will not be able to connect to the REST API exposed via IIS. ServiceInsight version 1.4 or greater is required to use Windows authentication.

Windows security, especially when used with custom LDAP configurations, can be tricky to set up properly. Some organizations may prefer to set up Basic authentication with credentials that are managed separately from the organization's domain. This would currently work for ServicePulse through the browser, but not for ServiceInsight.

The API client inside ServiceInsight should be enhanced to detect the 401 Unauthorized response and present a simple interface for providing a NetworkCredential that could be used for all future requests.

OpenID

ServiceControl can now be deployed to cloud services such as Azure Container Apps which allow layering OpenID authentication onto the ingress via a variety of services. While ServicePulse can follow the redirect, ServiceInsight is not able to authenticate using these providers.

Here is a spike PR where I attempted to sniff information from the WWW-Authenticate header and complete an OAuth login flow, but ultimately failed as there's not enough information in the WWW-Authenticate header to get the job done.

@AlexKeySmith
Copy link

I would also love to see some alternative authentication / authorisation methods supported out of the box.
We're looking to move away from AD auth as we lean on more PaaS. It would be great to support oAuth2 / Open ID Connect.

@DavidBoike DavidBoike changed the title Allow Basic authentication Additional authentication options Aug 30, 2024
@DavidBoike
Copy link
Member Author

I updated this issue to be more inclusive of other authentication options, and added a link to a spike I did to try OpenID authentication that (sadly) failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants