From 9928ca17eaa35aa50a074bc97740dfe4a8ba9d8f Mon Sep 17 00:00:00 2001 From: x032205 Date: Thu, 31 Jul 2025 12:51:56 -0400 Subject: [PATCH] Greptile review fixes --- .../cloudflare-connection-router.ts | 10 +++++----- .../acme/acme-certificate-authority-fns.ts | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/backend/src/server/routes/v1/app-connection-routers/cloudflare-connection-router.ts b/backend/src/server/routes/v1/app-connection-routers/cloudflare-connection-router.ts index 8a8d5deb1..e5344bb6d 100644 --- a/backend/src/server/routes/v1/app-connection-routers/cloudflare-connection-router.ts +++ b/backend/src/server/routes/v1/app-connection-routers/cloudflare-connection-router.ts @@ -72,8 +72,8 @@ export const registerCloudflareConnectionRouter = async (server: FastifyZodProvi handler: async (req) => { const { connectionId } = req.params; - const projects = await server.services.appConnection.cloudflare.listWorkersScripts(connectionId, req.permission); - return projects; + const scripts = await server.services.appConnection.cloudflare.listWorkersScripts(connectionId, req.permission); + return scripts; } }); @@ -99,9 +99,9 @@ export const registerCloudflareConnectionRouter = async (server: FastifyZodProvi onRequest: verifyAuth([AuthMode.JWT]), handler: async (req) => { const { connectionId } = req.params; - - const projects = await server.services.appConnection.cloudflare.listZones(connectionId, req.permission); - return projects; + s; + const zones = await server.services.appConnection.cloudflare.listZones(connectionId, req.permission); + return zones; } }); }; diff --git a/backend/src/services/certificate-authority/acme/acme-certificate-authority-fns.ts b/backend/src/services/certificate-authority/acme/acme-certificate-authority-fns.ts index 96e4a1292..86beadffe 100644 --- a/backend/src/services/certificate-authority/acme/acme-certificate-authority-fns.ts +++ b/backend/src/services/certificate-authority/acme/acme-certificate-authority-fns.ts @@ -410,7 +410,7 @@ export const AcmeCertificateAuthorityFns = ({ break; } default: { - throw new Error("Unsupported DNS provider"); + throw new Error(`Unsupported DNS provider: ${acmeCa.configuration.dnsProviderConfig.provider as string}`); } } }, @@ -438,7 +438,7 @@ export const AcmeCertificateAuthorityFns = ({ break; } default: { - throw new Error("Unsupported DNS provider"); + throw new Error(`Unsupported DNS provider: ${acmeCa.configuration.dnsProviderConfig.provider as string}`); } } }