refactor:resolved PR comments

Author: aishwarya-cstk

api/src/services/migration.service.ts

@@ -279,6 +279,14 @@ const deleteTestStack = async (req: Request): Promise<LoginServiceType> => {
 const startTestMigration = async (req: Request): Promise<any> => {
   const { orgId, projectId } = req?.params ?? {};
   const { region, user_id, is_sso } = req?.body?.token_payload ?? {};
+
+
+  if (is_sso !== true && is_sso !== false) {
+    throw new BadRequestError(
+      'Invalid token_payload.is_sso; expected a boolean value.',  
+    );
+  }
+
   await ProjectModelLowdb.read();
   const project: any = ProjectModelLowdb.chain
     .get('projects')
@@ -654,6 +662,13 @@ const startTestMigration = async (req: Request): Promise<any> => {
 const startMigration = async (req: Request): Promise<any> => {
   const { orgId, projectId } = req?.params ?? {};
   const { region, user_id, is_sso } = req?.body?.token_payload ?? {};
+
+  if (typeof is_sso !== 'boolean') {
+    throw new BadRequestError(
+      'Missing or invalid SSO flag in token payload: expected boolean "is_sso".',
+    );
+  }
+  
   await ProjectModelLowdb.read();
   const project: any = ProjectModelLowdb.chain
     .get('projects')

api/src/utils/content-type-creator.utils.ts

@@ -1026,8 +1026,27 @@ const writeGlobalField = async (schema: any, globalSave: string) => {
   }
 };
 
+const resolveIsSsoFlag = (is_sso: any): boolean => {
+  if (typeof is_sso === 'boolean') {
+    return is_sso;
+  }
+
+  if (is_sso === 'true') {
+    return true;
+  }
+
+  if (is_sso === 'false') {
+    return false;
+  }
+
+  throw new Error(
+    `Invalid token_payload.is_sso in existingCtMapper; expected boolean, received: ${JSON.stringify(is_sso)}`
+  );
+};
+
 const existingCtMapper = async ({ keyMapper, contentTypeUid, projectId, region, user_id, is_sso, type}: any) => {
   try {
+    const normalizedIsSso = resolveIsSsoFlag(is_sso);
     const ctUid = keyMapper?.[contentTypeUid];
 
     if(type === 'global_field') {
@@ -1041,7 +1060,7 @@ const existingCtMapper = async ({ keyMapper, contentTypeUid, projectId, region,
           token_payload: {
             region,
             user_id,
-            is_sso
+            is_sso: normalizedIsSso
           }
         }
       }
@@ -1057,7 +1076,7 @@ const existingCtMapper = async ({ keyMapper, contentTypeUid, projectId, region,
           token_payload: {
             region,
             user_id,
-            is_sso
+            is_sso: normalizedIsSso
           }
         }
       }