From 91209faab13636107764f94cdd31b06fcc213bca Mon Sep 17 00:00:00 2001 From: Joshen Lim Date: Tue, 9 Dec 2025 13:22:10 +0800 Subject: [PATCH] Remove unused old env var (#41178) * Remove unused old env var * Update apps/studio/lib/api/apiHelpers.ts Co-authored-by: Alaister Young * Fix * Add comment --------- Co-authored-by: Alaister Young --- apps/studio/lib/api/apiHelpers.test.ts | 1 - apps/studio/lib/api/apiHelpers.ts | 6 +++--- apps/studio/pages/api/platform/auth/[ref]/invite.ts | 3 --- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/apps/studio/lib/api/apiHelpers.test.ts b/apps/studio/lib/api/apiHelpers.test.ts index 5c2de3d0eb1..2caf0fdf386 100644 --- a/apps/studio/lib/api/apiHelpers.test.ts +++ b/apps/studio/lib/api/apiHelpers.test.ts @@ -15,7 +15,6 @@ vi.mock('lib/constants', () => ({ describe('apiHelpers', () => { describe('constructHeaders', () => { beforeEach(() => { - process.env.READ_ONLY_API_KEY = 'test-readonly-key' process.env.SUPABASE_SERVICE_KEY = 'test-service-key' }) diff --git a/apps/studio/lib/api/apiHelpers.ts b/apps/studio/lib/api/apiHelpers.ts index 2105fa452d9..b7a14da2d16 100644 --- a/apps/studio/lib/api/apiHelpers.ts +++ b/apps/studio/lib/api/apiHelpers.ts @@ -25,9 +25,9 @@ export function constructHeaders(headers: { [prop: string]: any }) { ) return { ...cleansedHeaders, - ...(IS_PLATFORM - ? { apiKey: `${process.env.READ_ONLY_API_KEY}` } - : { apiKey: `${process.env.SUPABASE_SERVICE_KEY}` }), + // [Joshen] JFYI both Alaister and I checked on this and realised this might not be used actually + // Could be safe to remove but leaving it here for now + ...(!IS_PLATFORM && { apiKey: `${process.env.SUPABASE_SERVICE_KEY}` }), } } else { return { diff --git a/apps/studio/pages/api/platform/auth/[ref]/invite.ts b/apps/studio/pages/api/platform/auth/[ref]/invite.ts index 662df08ded0..c72ac84d295 100644 --- a/apps/studio/pages/api/platform/auth/[ref]/invite.ts +++ b/apps/studio/pages/api/platform/auth/[ref]/invite.ts @@ -1,12 +1,9 @@ -import { createClient } from '@supabase/supabase-js' import { NextApiRequest, NextApiResponse } from 'next' import { fetchPost } from 'data/fetchers' import { constructHeaders } from 'lib/api/apiHelpers' import apiWrapper from 'lib/api/apiWrapper' -const supabase = createClient(process.env.SUPABASE_URL!, process.env.SUPABASE_SERVICE_KEY!) - export default (req: NextApiRequest, res: NextApiResponse) => apiWrapper(req, res, handler) async function handler(req: NextApiRequest, res: NextApiResponse) {