From 30bdc8b325d9255abd4ca81d07f2115c7ce43543 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Toma=C5=BE=20Jerman?= Date: Wed, 7 Dec 2022 12:32:27 +0100 Subject: [PATCH] Improve external auth provider BC by defaulting empty usage --- server/auth/handlers/handle_external.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/server/auth/handlers/handle_external.go b/server/auth/handlers/handle_external.go index 89912a673..89c2af590 100644 --- a/server/auth/handlers/handle_external.go +++ b/server/auth/handlers/handle_external.go @@ -112,7 +112,8 @@ func (h *AuthHandlers) handleSuccessfulExternalGenericAuth(w http.ResponseWriter au := request.GetAuthUser(h.SessionManager.Get(r)) // Check if we're using it as an identity provider; if so, use it to authenticate - if p.HasUsage(types.ExternalProviderUsageIdentity) { + // If no providers are specified, assume identity (legacy support) + if len(p.Usage) == 0 || p.HasUsage(types.ExternalProviderUsageIdentity) { // Try to login/sign-up external user if user, err = h.AuthService.External(ctx, cred); err != nil { handleErr(err)