rohit/probo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Log missing Crisp managed key before failing

Browse Source 
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>
This commit is contained in:
Aurélien Sibiril
2026-07-11 18:41:28 +02:00
parent 59e2d9a5df
commit 5b83911269
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/server/api/console/v1/connector_settings.go
View File

@@ -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. // here; treat its absence as an internal error rather than client input.
managedKey, ok := r.providerRegistry.ManagedAPIKey(input.Provider) managedKey, ok := r.providerRegistry.ManagedAPIKey(input.Provider)
if !ok { if !ok {
r.logger.ErrorCtx(ctx, "crisp managed api key not configured")
return gqlutils.Internal(ctx) return gqlutils.Internal(ctx)
} }