From a52a7456cee99c2b4a5d7e6900bb574a860ca135 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lucas=20Feij=C3=B3?= Date: Tue, 22 Oct 2024 15:16:49 -0300 Subject: [PATCH] fix: Avoid usage of `vtexinternal` host on development mode --- packages/api/src/platforms/vtex/clients/fetch.ts | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/packages/api/src/platforms/vtex/clients/fetch.ts b/packages/api/src/platforms/vtex/clients/fetch.ts index b75198a375..038759e9ae 100644 --- a/packages/api/src/platforms/vtex/clients/fetch.ts +++ b/packages/api/src/platforms/vtex/clients/fetch.ts @@ -3,6 +3,8 @@ import packageJson from '../../../../package.json' import { Options as ApiOptions } from '..' const USER_AGENT = `${packageJson.name}@${packageJson.version}` +const PRODUCTION_MODE = process.env.NODE_ENV === 'production' +const DEFAULT_PUBLIC_API_HOST = 'vtexcommercestable' const VTEX_API_TO_INTERNAL_API_NAME: { [key: string]: string } = { catalog: 'catalogapi', @@ -34,11 +36,12 @@ const getBasePrefix = async ({ environment: string vtexApi: RequestOptions['vtexApi'] }) => { - if (environment === 'vtexinternal') { - return `http://${VTEX_API_TO_INTERNAL_API_NAME[vtexApi]}.${environment}.com` + if (PRODUCTION_MODE && environment === 'vtexinternal') { + return `http://${VTEX_API_TO_INTERNAL_API_NAME[vtexApi]}.vtexinternal.com` } - return `https://${account}.${environment}.com.br` + // We should not use `environment` here when its value is `vtexinternal` + return `https://${account}.${environment === 'vtexinternal' ? DEFAULT_PUBLIC_API_HOST : environment}.com.br` } export const fetchAPI = async ({ path, init, options }: FetchAPI) => { @@ -58,7 +61,7 @@ export const fetchAPI = async ({ path, init, options }: FetchAPI) => { } // Check the environment to specify the account - if (options.environment === 'vtexinternal') { + if (PRODUCTION_MODE && options.environment === 'vtexinternal') { const hasParams = requestInfo.includes('?') requestInfo += hasParams