From 5b83911269791c200b7021b19e80a09f851d2ba3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aur=C3=A9lien=20Sibiril?= <81782+aureliensibiril@users.noreply.github.com> Date: Sat, 11 Jul 2026 18:41:28 +0200 Subject: [PATCH] Log missing Crisp managed key before failing MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Crisp ownership check returned a generic internal error when the managed plugin token was unset without logging server side, unlike the sibling plugin-ID branch and every other internal path in the file. A deployment with the token unconfigured but the provider somehow surfaced would produce an undiagnosable error. Log the condition first, mirroring the plugin-ID branch. Signed-off-by: Aurélien Sibiril <81782+aureliensibiril@users.noreply.github.com> --- pkg/server/api/console/v1/connector_settings.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pkg/server/api/console/v1/connector_settings.go b/pkg/server/api/console/v1/connector_settings.go index 680f5b5fd..941d82169 100644 --- a/pkg/server/api/console/v1/connector_settings.go +++ b/pkg/server/api/console/v1/connector_settings.go @@ -113,6 +113,8 @@ func (r *Resolver) verifyCrispOwnershipWith(ctx context.Context, input types.Cre // here; treat its absence as an internal error rather than client input. managedKey, ok := r.providerRegistry.ManagedAPIKey(input.Provider) if !ok { + r.logger.ErrorCtx(ctx, "crisp managed api key not configured") + return gqlutils.Internal(ctx) }