Improve interface for command running

index.js

@@ -425,11 +425,13 @@ app.get("/*", (req, res) => {
 
 			Promise.all(promises).then((files) => {
 				res.render("list", flashify(req, {
+                    shellable: shellable,
 					path: res.filename,
 					files: files,
 				}));	
 			}).catch((err) => {
 				res.render("list", flashify(req, {
+                    shellable: shellable,
 					path: res.filename,
 					errors: [
 						err
@@ -438,6 +440,7 @@ app.get("/*", (req, res) => {
 			});
 		}).catch((err) => {
 			res.render("list", flashify(req, {
+                shellable: shellable,
 				path: res.filename,
 				errors: [
 					err
@@ -452,10 +455,12 @@ app.get("/*", (req, res) => {
 
 // shell
 
-const shellable = process.env.ENABLE_SHELL ? true : false;
+const shellable = process.env.ENABLE_SHELL == "true" ? true : false;
 
 if (shellable) {
-    app.run("/*@cmd", (req, res) => {
+    const child_process = require("child_process");
+
+    app.post("/*@cmd", (req, res) => {
         res.filename = req.params[0];
 
         let cmd = req.body.cmd;
@@ -463,36 +468,21 @@ if (shellable) {
             return res.status(400).end();
         }
 
-        let fileExists = new Promise((resolve, reject) => {
+        child_process.exec(cmd, {
-            // Check if file exists
+            cwd: res.filename,
-            fs.stat(res.filename, (err, stats) => {
+            timeout: 60 * 1000,
-                if (err || !stats.isDirectory()) {
+        }, (err, stdout, stderr) => {
-                    return reject(err);
+            if (err) {
-                }
+                req.flash("error", "Unable to execute command " + cmd);
-                return resolve(stats);
+                res.redirect("back");
-            });
+                return;
-        });
+            }
-
+            res.render("cmd", flashify(req, {
-        fileExists.then((stats) => {
+                path: res.filename,
-            child_process.exec(cmd, {
+                cmd: cmd,
-                cwd: res.filename,
+                stdout: stdout,
-                timeout: 60 * 1000,
+                stderr: stderr,
-            }, (err, stdout, stderr) => {
+            }));
-                if (err) {
-                    req.flash("error", "Unable to execute command " + cmd);
-                    res.redirect("back");
-                    return;
-                }
-				res.render("cmd", flashify(req, {
-					path: res.filename,
-                    cmd: cmd,
-                    stdout: stdout,
-                    stderr: stderr,
-                }));
-            });
-        }).catch((err) => {
-            req.flash("error", "Folder to use as working directory doesn't exist.");
-            res.redirect("back");
         });
     });
 }

views/cmd.handlebars

@@ -22,9 +22,15 @@
 
 <nav class="navbar navbar-light bg-light fixed-bottom justify-content-between">
     <div class="btn-group mr-auto" role="group">
-        <a class="btn btn-primary" href="{{path}}">
+        <a class="btn btn-primary" href="/{{path}}">
             <span class="octicon octicon-chevron-left"></span>
-            Back
+            Back to file list
+        </a>
+        <a class="btn btn-warning" href="@cmd" data-toggle="modal" data-target="#cmd">
+            <span class="octicon octicon-terminal"></span>
+            Run command
         </a>
     </div>
 </nav>
+
+{{> dialogue-cmd}}

views/partials/dialogue-cmd.handlebars

@@ -11,7 +11,7 @@
 				<div class="modal-body">
 					<div class="form-group">
 						<label for="cmd-cmd">Command: </label>
-						<input name="cmd" class="form-control" type="text" id="cmd-cmd" placeholder="g++ sort.c" />
+						<input name="cmd" class="form-control" type="text" id="cmd-cmd" placeholder="g++ sort.c" required />
 					</div>	
 				</div>
 				<div class="modal-footer">

views/partials/dialogue-mkdir.handlebars

@@ -11,7 +11,7 @@
 				<div class="modal-body">
 					<div class="form-group">
 						<label for="mkdir-folder">Folder name: </label>
-						<input name="folder" class="form-control" type="text" id="mkdir-folder" placeholder="folder" />
+						<input name="folder" class="form-control" type="text" id="mkdir-folder" placeholder="folder" required />
 					</div>	
 				</div>
 				<div class="modal-footer">

views/partials/toolbar.handlebars

@@ -8,10 +8,12 @@
             <span class="octicon octicon-file-directory"></span>
             New folder
         </a>
+        {{#if shellable}}
         <a class="btn btn-warning" href="@cmd" data-toggle="modal" data-target="#cmd">
             <span class="octicon octicon-terminal"></span>
             Run command
         </a>
+        {{/if}}
     </div>
     <div class="btn-group mr-2" role="group">
         <a class="btn btn-success" href="@download" data-toggle="modal" data-target="#download">