From 3f8ea7764ec2fbd3dac99ac12d578aaff0078af8 Mon Sep 17 00:00:00 2001 From: lohit jiddimani Date: Thu, 17 Apr 2025 20:41:14 +0530 Subject: [PATCH] fix: add JSON parsing and error handling for Postman environment imports ~ return parsed JSON object instead of raw file string --- .../src/utils/importers/postman-environment.js | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/packages/bruno-app/src/utils/importers/postman-environment.js b/packages/bruno-app/src/utils/importers/postman-environment.js index a9cc22cbd..5afd0df57 100644 --- a/packages/bruno-app/src/utils/importers/postman-environment.js +++ b/packages/bruno-app/src/utils/importers/postman-environment.js @@ -6,7 +6,15 @@ const { postmanToBrunoEnvironment } = brunoConverters; const readFile = (files) => { return new Promise((resolve, reject) => { const fileReader = new FileReader(); - fileReader.onload = (e) => resolve(e.target.result); + fileReader.onload = (e) => { + try { + let parsedPostmanEnvironment = JSON.parse(e.target.result); + resolve(parsedPostmanEnvironment); + } catch (err) { + console.error(err); + reject(new BrunoError('Unable to parse the postman environment json file')); + } + } fileReader.onerror = (err) => reject(err); fileReader.readAsText(files[0]); });