Merge pull request #784 from akhilmhdh/fix/import-delete

fix: resolved secret import delete and include_import response control

backend/src/controllers/v1/secretImportController.ts

@@ -30,7 +30,6 @@ export const createSecretImport = async (req: Request, res: Response) => {
   if (doesImportExist) {
     throw BadRequestError({ message: "Secret import already exist" });
   }
-
   importSecDoc.imports.push({
     environment: secretImport.environment,
     secretPath: secretImport.secretPath

backend/src/controllers/v2/secretsController.ts

@@ -830,7 +830,7 @@ export const getSecrets = async (req: Request, res: Response) => {
 
   // TODO(akhilmhdh) - secret-imp change this to org type
   let importedSecrets: any[] = [];
-  if (include_imports) {
+  if (include_imports === "true") {
     importedSecrets = await getAllImportedSecrets(workspaceId, environment, folderId as string);
   }
 

backend/src/controllers/v3/secretsController.ts

@@ -26,13 +26,16 @@ export const getSecretsRaw = async (req: Request, res: Response) => {
   const includeImports = req.query.include_imports as string;
 
   // if the service token has single scope, it will get all secrets for that scope by default
-  const serviceTokenDetails: IServiceTokenData = req?.serviceTokenData
+  const serviceTokenDetails: IServiceTokenData = req?.serviceTokenData;
   if (serviceTokenDetails) {
-    if (serviceTokenDetails.scopes.length == 1 && !containsGlobPatterns(serviceTokenDetails.scopes[0].secretPath)) {
+    if (
-      const scope = serviceTokenDetails.scopes[0]
+      serviceTokenDetails.scopes.length == 1 &&
-      secretPath = scope.secretPath
+      !containsGlobPatterns(serviceTokenDetails.scopes[0].secretPath)
-      environment = scope.environment
+    ) {
-      workspaceId = serviceTokenDetails.workspace.toString()
+      const scope = serviceTokenDetails.scopes[0];
+      secretPath = scope.secretPath;
+      environment = scope.environment;
+      workspaceId = serviceTokenDetails.workspace.toString();
     } else {
       requireWorkspaceAuth({
         acceptedRoles: [ADMIN, MEMBER],
@@ -41,7 +44,7 @@ export const getSecretsRaw = async (req: Request, res: Response) => {
         requiredPermissions: [PERMISSION_READ_SECRETS],
         requireBlindIndicesEnabled: true,
         requireE2EEOff: true
-      })
+      });
     }
   }
 
@@ -294,7 +297,7 @@ export const getSecrets = async (req: Request, res: Response) => {
     authData: req.authData
   });
 
-  if (includeImports) {
+  if (includeImports === "true") {
     const folders = await Folder.findOne({ workspace: workspaceId, environment });
     let folderId = "root";
     // if folder exist get it and replace folderid with new one

backend/src/routes/v1/secretImport.ts

@@ -46,7 +46,7 @@ router.delete(
   body("secretImportPath").isString().exists().trim(),
   body("secretImportEnv").isString().exists().trim(),
   validateRequest,
-  secretImportController.updateSecretImport
+  secretImportController.deleteSecretImport
 );
 
 router.get(