Skip to content

Commit 12fbc4c

Browse files
fix: extend SageMaker env detection to support both SMUS and SMAI (#2598)
* fix: remove env variable check to support SMAI * fix: extend SageMaker env detection to support both SMUS and SMAI --------- Co-authored-by: invictus <149003065+ashishrp-aws@users.noreply.github.com>
1 parent 55f4c34 commit 12fbc4c

File tree

1 file changed

+13
-2
lines changed
  • server/aws-lsp-codewhisperer/src/shared

1 file changed

+13
-2
lines changed

server/aws-lsp-codewhisperer/src/shared/utils.ts

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -361,9 +361,20 @@ export function getBearerTokenFromProviderWithType(credentialsProvider: Credenti
361361

362362
return credentials.token
363363
}
364+
function isSagemakerEnv(): boolean {
365+
return (
366+
process.env.SAGEMAKER_APP_TYPE !== undefined ||
367+
process.env.SAGEMAKER_INTERNAL_IMAGE_URI !== undefined ||
368+
process.env.STUDIO_LOGGING_DIR?.includes('/var/log/studio') === true ||
369+
process.env.SM_APP_TYPE !== undefined ||
370+
process.env.SM_INTERNAL_IMAGE_URI !== undefined ||
371+
process.env.SERVICE_NAME === 'SageMakerUnifiedStudio'
372+
)
373+
}
374+
364375
export function getClientName(lspParams: InitializeParams | undefined): string | undefined {
365-
return process.env.SERVICE_NAME === SAGEMAKER_UNIFIED_STUDIO_SERVICE
366-
? lspParams?.initializationOptions?.aws?.clientInfo?.name
376+
return isSagemakerEnv()
377+
? lspParams?.initializationOptions?.aws?.clientInfo?.name || lspParams?.clientInfo?.name
367378
: lspParams?.clientInfo?.name
368379
}
369380

0 commit comments

Comments
 (0)