The exec command only returns success output for now

StanleyMasinde · StanleyMasinde · commit 9c20a6ccba0c · 2020-11-22T17:59:23.000+03:00
* Proper handling of errors will be added
 Changes to be committed:
	modified:   web/app/controllers/ConfigController.js
diff --git a/web/app/controllers/ConfigController.js b/web/app/controllers/ConfigController.js
@@ -96,10 +96,10 @@ module.exports = new class ConfigController extends Controller {
             const runCommad = new Promise((resolve, _reject) => {
                 exec(cfg.command, (error, stdout, stderr) => {
                     if (error) {
-                        return resolve(error)
+                        // TODO handle this properly
                     }
                     if (stderr) {
-                        return resolve(stderr)
+                        // TODO  Handle warninngs properly
                     }
                     return resolve(stdout)
                 })

Original file line number	Diff line number	Diff line change
`@@ -96,10 +96,10 @@ module.exports = new class ConfigController extends Controller {`
`96`	`96`	`const runCommad = new Promise((resolve, _reject) => {`
`97`	`97`	`exec(cfg.command, (error, stdout, stderr) => {`
`98`	`98`	`if (error) {`
`99`		`- return resolve(error)`
	`99`	`+ // TODO handle this properly`
`100`	`100`	`}`
`101`	`101`	`if (stderr) {`
`102`		`- return resolve(stderr)`
	`102`	`+ // TODO Handle warninngs properly`
`103`	`103`	`}`
`104`	`104`	`return resolve(stdout)`
`105`	`105`	`})`