From 91a06d1fb578cb7e12f8bbaebbabaa32ac1632e7 Mon Sep 17 00:00:00 2001 From: Bryan Frimin Date: Tue, 17 Mar 2026 14:31:23 +0100 Subject: [PATCH] Use logger + gqlutils.Internal pattern instead of panic for NDA errors Signed-off-by: Bryan Frimin --- pkg/server/api/console/v1/v1_resolver.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/server/api/console/v1/v1_resolver.go b/pkg/server/api/console/v1/v1_resolver.go index d82866aaf..ae905c7dc 100644 --- a/pkg/server/api/console/v1/v1_resolver.go +++ b/pkg/server/api/console/v1/v1_resolver.go @@ -6766,16 +6766,16 @@ func (r *organizationResolver) TrustCenter(ctx context.Context, obj *types.Organ trustCenter, err := prb.TrustCenters.GetByOrganizationID(ctx, obj.ID) if err != nil { - // TODO no panic use gqlutils.InternalError - panic(fmt.Errorf("cannot get trust center: %w", err)) + r.logger.ErrorCtx(ctx, "cannot get trust center", log.Error(err)) + return nil, gqlutils.Internal(ctx) } var file *coredata.File if trustCenter.NonDisclosureAgreementFileID != nil { file, err = prb.Files.Get(ctx, *trustCenter.NonDisclosureAgreementFileID) if err != nil { - // TODO no panic use gqlutils.InternalError - panic(fmt.Errorf("cannot get NDA file: %w", err)) + r.logger.ErrorCtx(ctx, "cannot get NDA file", log.Error(err)) + return nil, gqlutils.Internal(ctx) } }