.github/workflows/package-pr.yaml

@@ -71,7 +71,7 @@ jobs:
           docker run -v $(pwd)'/repo:/piston/repo' -v $(pwd)'/packages:/piston/packages' -d --name repo docker.pkg.github.com/engineer-man/piston/repo-builder --no-build
           docker run --network container:repo -v $(pwd)'/data:/piston' -e PISTON_LOG_LEVEL=DEBUG -e 'PISTON_REPO_URL=http://localhost:8000/index' -d --name api docker.pkg.github.com/engineer-man/piston/api
           echo Waiting for API to start..
-          docker run --network container:api appropriate/curl -s --retry 10 --retry-connrefused http://localhost:2000/api/v2/runtimes
+          docker run --network container:api appropriate/curl -s --retry 10 --retry-connrefused http://localhost:2000/api/v1/runtimes
 
           echo Waiting for Index to start..
           docker run --network container:repo appropriate/curl -s --retry 999 --retry-max-time 0 --retry-connrefused http://localhost:8000/index
@@ -80,7 +80,7 @@ jobs:
           sed -i 's/repo/localhost/g' repo/index
 
           echo Listing Packages
-          PACKAGES_JSON=$(docker run --network container:api appropriate/curl -s http://localhost:2000/api/v2/packages)
+          PACKAGES_JSON=$(docker run --network container:api appropriate/curl -s http://localhost:2000/api/v1/packages)
           echo $PACKAGES_JSON
 
           echo Getting CLI ready
@@ -94,7 +94,7 @@ jobs:
             PKG_VERSION=$(awk -F- '{ print $2 }' <<< $package)
 
             echo "Installing..."
-            docker run --network container:api appropriate/curl -sXPOST http://localhost:2000/api/v2/packages/$PKG_PATH
+            docker run --network container:api appropriate/curl -sXPOST http://localhost:2000/api/v1/packages/$PKG_PATH
 
             TEST_SCRIPTS=packages/$PKG_PATH/test.*
             echo "Tests: $TEST_SCRIPTS"

api/.prettierignore

@@ -1 +0,0 @@
-node_modules

api/.prettierrc.yaml

@@ -1,3 +0,0 @@
-singleQuote: true
-tabWidth: 4
-arrowParens: avoid

api/Dockerfile

@@ -27,3 +27,4 @@ RUN make -C ./src/nosocket/ all && make -C ./src/nosocket/ install
 
 CMD [ "node", "src"]
 EXPOSE 2000/tcp
+

api/package-lock.json

@@ -19,9 +19,6 @@
                 "semver": "^7.3.4",
                 "uuid": "^8.3.2",
                 "waitpid": "git+https://github.com/HexF/node-waitpid.git"
-            },
-            "devDependencies": {
-                "prettier": "2.2.1"
             }
         },
         "node_modules/accepts": {
@@ -394,18 +391,6 @@
             "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-0.1.7.tgz",
             "integrity": "sha1-32BBeABfUi8V60SQ5yR6G/qmf4w="
         },
-        "node_modules/prettier": {
-            "version": "2.2.1",
-            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.2.1.tgz",
-            "integrity": "sha512-PqyhM2yCjg/oKkFPtTGUojv7gnZAoG80ttl45O6x2Ug/rMJw4wcc9k6aaf2hibP7BGVCCM33gZoGjyvt9mm16Q==",
-            "dev": true,
-            "bin": {
-                "prettier": "bin-prettier.js"
-            },
-            "engines": {
-                "node": ">=10.13.0"
-            }
-        },
         "node_modules/proxy-addr": {
             "version": "2.0.6",
             "resolved": "https://registry.npmjs.org/proxy-addr/-/proxy-addr-2.0.6.tgz",
@@ -870,12 +855,6 @@
             "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-0.1.7.tgz",
             "integrity": "sha1-32BBeABfUi8V60SQ5yR6G/qmf4w="
         },
-        "prettier": {
-            "version": "2.2.1",
-            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.2.1.tgz",
-            "integrity": "sha512-PqyhM2yCjg/oKkFPtTGUojv7gnZAoG80ttl45O6x2Ug/rMJw4wcc9k6aaf2hibP7BGVCCM33gZoGjyvt9mm16Q==",
-            "dev": true
-        },
         "proxy-addr": {
             "version": "2.0.6",
             "resolved": "https://registry.npmjs.org/proxy-addr/-/proxy-addr-2.0.6.tgz",

api/package.json

@@ -15,11 +15,5 @@
         "uuid": "^8.3.2",
         "waitpid": "git+https://github.com/HexF/node-waitpid.git"
     },
-    "license": "MIT",
+    "license": "MIT"
-    "scripts": {
-        "lint": "prettier . --write"
-    },
-    "devDependencies": {
-        "prettier": "2.2.1"
-    }
 }

api/src/api/v2.js

@@ -3,110 +3,101 @@ const router = express.Router();
 
 const config = require('../config');
 const runtime = require('../runtime');
-const { Job } = require('../job');
+const {Job} = require("../job");
-const package = require('../package');
+const package = require('../package')
-const logger = require('logplease').create('api/v2');
+const logger = require('logplease').create('api/v1');
-
-router.use((req, res, next) => {
-    if (['GET', 'HEAD', 'OPTIONS'].includes(req.method)) {
-        return next();
-    }
-
-    if (!req.headers['content-type'].startsWith('application/json')) {
-        return res.status(415).send({
-            message: 'requests must be of type application/json',
-        });
-    }
 
+router.use(function(req, res, next){
+    if(req.method == "POST" && !req.headers['content-type'].startsWith("application/json"))
+        return res
+            .status(415)
+            .send({
+                message: "requests must be of type application/json"
+            })
     next();
-});
+})
 
-router.post('/execute', async (req, res) => {
+router.post('/execute', async function(req, res){
-    const {
+        const {language, version, files, stdin, args, run_timeout, compile_timeout, compile_memory_limit, run_memory_limit} = req.body;
-        language,
-        version,
-        files,
-        stdin,
-        args,
-        run_timeout,
-        compile_timeout,
-        compile_memory_limit,
-        run_memory_limit,
-    } = req.body;
 
-    if (!language || typeof language !== 'string') {
+        if(!language || typeof language !== "string")
-        return res.status(400).send({
+        {
-            message: 'language is required as a string',
+            return res
+                .status(400)
+                .send({
+                    message: "language is required as a string"
                 });
         }
 
-    if (!version || typeof version !== 'string') {
+        if(!version || typeof version !== "string")
-        return res.status(400).send({
+        {
-            message: 'version is required as a string',
+            return res
+                .status(400)
+                .send({
+                    message: "version is required as a string"
                 });
         }
 
-    if (!files || !Array.isArray(files)) {
+        if(!files || !Array.isArray(files))
-        return res.status(400).send({
+        {
-            message: 'files is required as an array',
+            return res
+                .status(400)
+                .send({
+                    message: "files is required as an array"
                 });
         }
 
         for (const [i,file] of files.entries()) {
-        if (typeof file.content !== 'string') {
+            if(typeof file.content !== "string"){
-            return res.status(400).send({
+                return res
-                message: `files[${i}].content is required as a string`,
+                    .status(400)
+                    .send({
+                        message: `files[${i}].content is required as a string`
                     });
             }
         }
 
         if (compile_memory_limit) {
-        if (typeof compile_memory_limit !== 'number') {
+            if (typeof compile_memory_limit !== "number") {
-            return res.status(400).send({
+                return res
-                message: 'if specified, compile_memory_limit must be a number',
+                    .status(400)
-            });
+                    .send({
-        }
+                        message: "if specified, compile_memory_limit must be a number"
-
+                    })
-        if (
+            } else if (config.compile_memory_limit >= 0 && (compile_memory_limit > config.compile_memory_limit || compile_memory_limit < 0)) {
-            config.compile_memory_limit >= 0 &&
+                return res
-            (compile_memory_limit > config.compile_memory_limit ||
+                    .status(400)
-                compile_memory_limit < 0)
+                    .send({
-        ) {
+                        message: "compile_memory_limit cannot exceed the configured limit of " + config.compile_memory_limit
-            return res.status(400).send({
+                    })
-                message:
-                    'compile_memory_limit cannot exceed the configured limit of ' +
-                    config.compile_memory_limit,
-            });
             }
         }
 
         if (run_memory_limit) {
-        if (typeof run_memory_limit !== 'number') {
+            if (typeof run_memory_limit !== "number") {
-            return res.status(400).send({
+                return res
-                message: 'if specified, run_memory_limit must be a number',
+                    .status(400)
-            });
+                    .send({
-        }
+                        message: "if specified, run_memory_limit must be a number"
-
+                    })
-        if (
+            } else if (config.run_memory_limit >= 0 && (run_memory_limit > config.run_memory_limit || run_memory_limit < 0)) {
-            config.run_memory_limit >= 0 &&
+                return res
-            (run_memory_limit > config.run_memory_limit || run_memory_limit < 0)
+                    .status(400)
-        ) {
+                    .send({
-            return res.status(400).send({
+                        message: "run_memory_limit cannot exceed the configured limit of " + config.run_memory_limit
-                message:
+                    })
-                    'run_memory_limit cannot exceed the configured limit of ' +
-                    config.run_memory_limit,
-            });
             }
         }
 
-    const rt = runtime.get_latest_runtime_matching_language_version(
+
-        language,
+    
-        version
+        const rt = runtime.get_latest_runtime_matching_language_version(language, version);
-    );
 
         if (rt === undefined) {
-        return res.status(400).send({
+            return res
-            message: `${language}-${version} runtime is unknown`,
+                .status(400)
+                .send({
+                    message: `${language}-${version} runtime is unknown`
                 });
         }
 
@@ -115,15 +106,15 @@ router.post('/execute', async (req, res) => {
             alias: language,
             files: files,
             args: args || [],
-        stdin: stdin || '',
+            stdin: stdin || "",
             timeouts: {
                 run: run_timeout || 3000,
-            compile: compile_timeout || 10000,
+                compile: compile_timeout || 10000
             },
             memory_limits: {
                 run: run_memory_limit || config.run_memory_limit,
-            compile: compile_memory_limit || config.compile_memory_limit,
+                compile: compile_memory_limit || config.compile_memory_limit
-        },
+            }
         });
 
         await job.prime();
@@ -132,93 +123,108 @@ router.post('/execute', async (req, res) => {
 
         await job.cleanup();
 
-    return res.status(200).send(result);
+        return res
+            .status(200)
+            .send(result);
 });
 
-router.get('/runtimes', (req, res) => {
+router.get('/runtimes', function(req, res){
-    const runtimes = runtime.map(rt => {
+    const runtimes = runtime.map(rt => ({
-        return {
         language: rt.language,
         version: rt.version.raw,
         aliases: rt.aliases,
-            runtime: rt.runtime,
+        runtime: rt.runtime
-        };
+    }));
+
+    return res
+        .status(200)
+        .send(runtimes);
 });
 
-    return res.status(200).send(runtimes);
+router.get('/packages', async function(req, res){
-});
-
-router.get('/packages', async (req, res) => {
     logger.debug('Request to list packages');
     let packages = await package.get_package_list();
 
-    packages = packages.map(pkg => {
+    packages = packages
+        .map(pkg => {
                 return {
                     language: pkg.language,
                     language_version: pkg.version.raw,
-            installed: pkg.installed,
+                    installed: pkg.installed
                 };
             });
 
-    return res.status(200).send(packages);
+    return res
+        .status(200)
+        .send(packages);
 });
 
-router.post('/packages', async (req, res) => {
+router.post('/packages/:language/:version', async function(req, res){
     logger.debug('Request to install package');
 
-    const { language, version } = req.body;
+    const {language, version} = req.params;
 
     const pkg = await package.get_package(language, version);
 
     if (pkg == null) {
-        return res.status(404).send({
+        return res
-            message: `Requested package ${language}-${version} does not exist`,
+            .status(404)
+            .send({
+                message: `Requested package ${language}-${version} does not exist`
             });
     }
 
     try {
         const response = await pkg.install();
 
-        return res.status(200).send(response);
+        return res
+            .status(200)
+            .send(response);
     } catch(e) {
-        logger.error(
+        logger.error(`Error while installing package ${pkg.language}-${pkg.version}:`, e.message);
-            `Error while installing package ${pkg.language}-${pkg.version}:`,
-            e.message
-        );
 
-        return res.status(500).send({
+        return res
-            message: e.message,
+            .status(500)
+            .send({
+                message: e.message
             });
     }
 });
 
-router.delete('/packages', async (req, res) => {
+router.delete('/packages/:language/:version', async function(req, res){
     logger.debug('Request to uninstall package');
 
-    const { language, version } = req.body;
+    const {language, version} = req.params;
 
     const pkg = await package.get_package(language, version);
 
     if (pkg == null) {
-        return res.status(404).send({
+        return res
-            message: `Requested package ${language}-${version} does not exist`,
+            .status(404)
+            .send({
+                message: `Requested package ${language}-${version} does not exist`
             });
     }
 
     try {
         const response = await pkg.uninstall();
 
-        return res.status(200).send(response);
+        return res
+            .status(200)
+            .send(response);
     } catch(e) {
-        logger.error(
+        logger.error(`Error while uninstalling package ${pkg.language}-${pkg.version}:`, e.message);
-            `Error while uninstalling package ${pkg.language}-${pkg.version}:`,
-            e.message
-        );
 
-        return res.status(500).send({
+        return res
-            message: e.message,
+            .status(500)
+            .send({
+                message: e.message
             });
     }
 });
 
+
+
+
+
 module.exports = router;

api/src/config.js

@@ -2,6 +2,7 @@ const fss = require('fs');
 const Logger = require('logplease');
 const logger = Logger.create('config');
 
+
 const options = [
     {
         key: 'log_level',
@@ -9,111 +10,126 @@ const options = [
         default: 'INFO',
         options: Object.values(Logger.LogLevels),
         validators: [
-            x =>
+            x => Object.values(Logger.LogLevels).includes(x) || `Log level ${x} does not exist`
-                Object.values(Logger.LogLevels).includes(x) ||
+        ]
-                `Log level ${x} does not exist`,
-        ],
     },
     {
         key: 'bind_address',
-        desc: 'Address to bind REST API on',
+        desc: 'Address to bind REST API on\nThank @Bones for the number',
         default: '0.0.0.0:2000',
-        validators: [],
+        validators: []
     },
     {
         key: 'data_directory',
         desc: 'Absolute path to store all piston related data at',
         default: '/piston',
-        validators: [
+        validators: [x=> fss.exists_sync(x) || `Directory ${x} does not exist`]
-            x => fss.exists_sync(x) || `Directory ${x} does not exist`,
-        ],
     },
     {
         key: 'runner_uid_min',
         desc: 'Minimum uid to use for runner',
         default: 1001,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'runner_uid_max',
         desc: 'Maximum uid to use for runner',
         default: 1500,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'runner_gid_min',
         desc: 'Minimum gid to use for runner',
         default: 1001,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'runner_gid_max',
         desc: 'Maximum gid to use for runner',
         default: 1500,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'disable_networking',
         desc: 'Set to true to disable networking',
         default: true,
-        parser: x => x === 'true',
+        parser: x => x === "true",
-        validators: [x => typeof x === 'boolean' || `${x} is not a boolean`],
+        validators: [
+            x => typeof x === "boolean" || `${x} is not a boolean`
+        ]
     },
     {
         key: 'output_max_size',
         desc: 'Max size of each stdio buffer',
         default: 1024,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'max_process_count',
         desc: 'Max number of processes per job',
         default: 64,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'max_open_files',
         desc: 'Max number of open files per job',
         default: 2048,
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'max_file_size',
         desc: 'Max file size in bytes for a file',
-        default: 10000000, //10MB
+        default: 1000000, //1MB
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'compile_memory_limit',
-        desc:
+        desc: 'Max memory usage for compile stage in bytes (set to -1 for no limit)',
-            'Max memory usage for compile stage in bytes (set to -1 for no limit)',
         default: -1, // no limit
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'run_memory_limit',
-        desc:
+        desc: 'Max memory usage for run stage in bytes (set to -1 for no limit)',
-            'Max memory usage for run stage in bytes (set to -1 for no limit)',
         default: -1, // no limit
         parser: parse_int,
-        validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
+        validators: [
+            (x,raw) => !isNaN(x) || `${raw} is not a number`,
+        ]
     },
     {
         key: 'repo_url',
         desc: 'URL of repo index',
-        default:
+        default: 'https://github.com/engineer-man/piston/releases/download/pkgs/index',
-            'https://github.com/engineer-man/piston/releases/download/pkgs/index',
+        validators: []
-        validators: [],
+    }
-    },
 ];
 
 logger.info(`Loading Configuration from environment`);
@@ -123,7 +139,7 @@ let errored = false;
 let config = {};
 
 options.forEach(option => {
-    const env_key = 'PISTON_' + option.key.to_upper_case();
+    const env_key = "PISTON_" + option.key.to_upper_case();
 
     const parser = option.parser || (x=>x);
 
@@ -131,19 +147,20 @@ options.forEach(option => {
 
     const parsed_val = parser(env_val);
 
+    
     const value = env_val || option.default;
 
+
     option.validators.for_each(validator => {
         let response = null;
-        if (env_val) response = validator(parsed_val, env_val);
+        if(env_val)
-        else response = validator(value, value);
+            response = validator(parsed_val, env_val);
+        else
+            response = validator(value, value);
 
         if (response !== true) {
             errored = true;
-            logger.error(
+            logger.error(`Config option ${option.key} failed validation:`, response);
-                `Config option ${option.key} failed validation:`,
-                response
-            );
             return;
         }
     });
@@ -157,4 +174,5 @@ if (errored) {
 
 logger.info('Configuration successfully loaded');
 
+
 module.exports = config;

api/src/globals.js

@@ -1,20 +1,26 @@
 // Globals are things the user shouldn't change in config, but is good to not use inline constants for
 const is_docker = require('is-docker');
-const fs = require('fs');
+const fss = require('fs');
-const platform = `${is_docker() ? 'docker' : 'baremetal'}-${fs
+const platform = `${is_docker() ? 'docker' : 'baremetal'}-${
-    .read_file_sync('/etc/os-release')
+    fss.read_file_sync('/etc/os-release')
         .toString()
         .split('\n')
         .find(x => x.startsWith('ID'))
-    .replace('ID=', '')}`;
+        .replace('ID=','')
+}`;
 
 module.exports = {
     data_directories: {
         packages: 'packages',
-        jobs: 'jobs',
+        jobs: 'jobs'
     },
     version: require('../package.json').version,
     platform,
     pkg_installed_file: '.ppman-installed', //Used as indication for if a package was installed
-    clean_directories: ['/dev/shm', '/run/lock', '/tmp', '/var/tmp'],
+    clean_directories: [
+        "/dev/shm",
+        "/run/lock",
+        "/tmp",
+        "/var/tmp"
+    ]
 };

api/src/index.js

@@ -35,28 +35,21 @@ const app = express();
     });
 
     logger.info('Loading packages');
-    const pkgdir = path.join(
+    const pkgdir = path.join(config.data_directory,globals.data_directories.packages);
-        config.data_directory,
-        globals.data_directories.packages
-    );
 
     const pkglist = await fs.readdir(pkgdir);
 
     const languages = await Promise.all(
-        pkglist.map(lang => {
+        pkglist.map(lang=>
-            return fs.readdir(path.join(pkgdir, lang)).then(x => {
+            fs.readdir(path.join(pkgdir,lang))
-                return x.map(y => path.join(pkgdir, lang, y));
+                .then(x=>x.map(y=>path.join(pkgdir, lang, y)))
-            });
+        ));
-        })
-    );
 
     const installed_languages = languages
         .flat()
-        .filter(pkg =>
+        .filter(pkg => fss.exists_sync(path.join(pkg, globals.pkg_installed_file)));
-            fss.exists_sync(path.join(pkg, globals.pkg_installed_file))
-        );
 
-    installed_languages.for_each(pkg => runtime.load_package(pkg));
+    installed_languages.forEach(pkg => runtime.load_package(pkg));
 
     logger.info('Starting API Server');
     logger.debug('Constructing Express App');
@@ -65,19 +58,21 @@ const app = express();
     app.use(body_parser.urlencoded({ extended: true }));
     app.use(body_parser.json());
 
-    app.use((err, req, res, next) => {
+    app.use(function (err, req, res, next) {
-        return res.status(400).send({
+        return res
-            stack: err.stack,
+            .status(400)
-        });
+            .send({
-    });
+                stack: err.stack
+            })
+      })
 
     logger.debug('Registering Routes');
 
-    const api_v2 = require('./api/v2');
+    const api_v2 = require('./api/v2')
-    app.use('/api/v2', api_v2);
+    app.use('/api/v1', api_v2); 
     app.use('/api/v2', api_v2); 
 
-    app.use((req, res, next) => {
+    app.use(function (req,res,next){
         return res.status(404).send({message: 'Not Found'});
     });
 

api/src/job.js

@@ -10,19 +10,20 @@ const wait_pid = require('waitpid');
 const job_states = {
     READY: Symbol('Ready to be primed'),
     PRIMED: Symbol('Primed and ready for execution'),
-    EXECUTED: Symbol('Executed and ready for cleanup'),
+    EXECUTED: Symbol('Executed and ready for cleanup')
 };
 
 let uid = 0;
 let gid = 0;
 
 class Job {
+
     constructor({ runtime, files, args, stdin, timeouts, memory_limits }) {
         this.uuid =  uuidv4();
         this.runtime = runtime;
         this.files = files.map((file,i) => ({
             name: file.name || `file${i}.code`,
-            content: file.content,
+            content: file.content
         }));
         
         this.args = args;
@@ -36,15 +37,12 @@ class Job {
         uid++;
         gid++;
 
-        uid %= config.runner_uid_max - config.runner_uid_min + 1;
+        uid %= (config.runner_uid_max - config.runner_uid_min) + 1;
-        gid %= config.runner_gid_max - config.runner_gid_min + 1;
+        gid %= (config.runner_gid_max - config.runner_gid_min) + 1;
+
 
         this.state = job_states.READY;
-        this.dir = path.join(
+        this.dir = path.join(config.data_directory, globals.data_directories.jobs, this.uuid);
-            config.data_directory,
-            globals.data_directories.jobs,
-            this.uuid
-        );
     }
 
     async prime() {
@@ -77,14 +75,19 @@ class Job {
                 'prlimit',
                 '--nproc=' + config.max_process_count,
                 '--nofile=' + config.max_open_files,
-                '--fsize=' + config.max_file_size,
+                '--fsize=' + config.max_file_size
             ];
 
             if (memory_limit >= 0) {
                 prlimit.push('--as=' + memory_limit);
             }
 
-            const proc_call = [...prlimit, ...nonetwork, 'bash', file, ...args];
+            const proc_call = [
+                ...prlimit,
+                ...nonetwork,
+                'bash',file,
+                ...args
+            ];
 
             var stdout = '';
             var stderr = '';
@@ -93,23 +96,20 @@ class Job {
             const proc = cp.spawn(proc_call[0], proc_call.splice(1) ,{
                 env: { 
                     ...this.runtime.env_vars,
-                    PISTON_LANGUAGE: this.runtime.language,
+                    PISTON_LANGUAGE: this.runtime.language
                 },
                 stdio: 'pipe',
                 cwd: this.dir,
                 uid: this.uid,
                 gid: this.gid,
-                detached: true, //give this process its own process group
+                detached: true //give this process its own process group
             });
 
             proc.stdin.write(this.stdin);
             proc.stdin.end();
             proc.stdin.destroy();
 
-            const kill_timeout = set_timeout(
+            const kill_timeout = set_timeout(_ => proc.kill('SIGKILL'), timeout);
-                _ => proc.kill('SIGKILL'),
-                timeout
-            );
 
             proc.stderr.on('data', data => {
                 if (stderr.length > config.output_max_size) {
@@ -142,7 +142,7 @@ class Job {
                 resolve({ stdout, stderr, code, signal, output });
             });
 
-            proc.on('error', err => {
+            proc.on('error', (err) => {
                 exit_cleanup();
 
                 reject({ error: err, stdout, stderr, output });
@@ -152,17 +152,10 @@ class Job {
 
     async execute() {
         if (this.state !== job_states.PRIMED) {
-            throw new Error(
+            throw new Error('Job must be in primed state, current state: ' + this.state.toString());
-                'Job must be in primed state, current state: ' +
-                    this.state.toString()
-            );
         }
 
-        logger.info(
+        logger.info(`Executing job uuid=${this.uuid} uid=${this.uid} gid=${this.gid} runtime=${this.runtime.toString()}`);
-            `Executing job uuid=${this.uuid} uid=${this.uid} gid=${
-                this.gid
-            } runtime=${this.runtime.toString()}`
-        );
 
         logger.debug('Compiling');
 
@@ -192,16 +185,15 @@ class Job {
             compile,
             run,
             language: this.runtime.language,
-            version: this.runtime.version.raw,
+            version: this.runtime.version.raw
         };
     }
 
+
     async cleanup_processes(){
         let processes = [1];
-
         while(processes.length > 0){
-            processes = await new Promise((resolve, reject) =>
+            processes = await new Promise((resolve, reject) => cp.execFile('ps', ['awwxo', 'pid,ruid'], function(err, stdout) {
-                cp.execFile('ps', ['awwxo', 'pid,ruid'], (err, stdout) => {
                 if(err === null){
                     const lines = stdout.split('\n').slice(1); //Remove header with slice
                     const procs = lines.map(line => {
@@ -210,17 +202,16 @@ class Job {
                             .split(/\s+/)
                             .map(n => parseInt(n));
 
-                            return { pid, ruid };
+                        return { pid, ruid }
-                        });
-
-                        resolve(procs);
-                    } else {
-                        reject(error);
-                    }
                     })
-            );
+                    resolve(procs)
+                }
+                else{
+                    reject(error)
+                }
+            }));
 
-            processes = processes.filter(proc => proc.ruid === this.uid);
+            processes = processes.filter(proc => proc.ruid == this.uid);
 
             for(const proc of processes){
                 // First stop the processes, but keep their resources allocated so they cant re-fork
@@ -231,6 +222,7 @@ class Job {
                 }
             }
 
+
             for(const proc of processes){
                 // Then clear them out of the process tree
                 try{
@@ -238,33 +230,28 @@ class Job {
                 }catch{
                     // Could already be dead and just needs to be waited on
                 }
-
                 wait_pid(proc.pid);
             }
         }
     }
 
     async cleanup_filesystem(){
+        
         for (const clean_path of globals.clean_directories) {
             const contents = await fs.readdir(clean_path);
 
             for (const file of contents) {
                 const file_path = path.join(clean_path, file);
-
                 try{
                     const stat = await fs.stat(file_path);
-
+                    if(stat.uid == this.uid)
-                    if (stat.uid === this.uid) {
+                        await fs.rm(file_path,  { recursive: true, force: true });
-                        await fs.rm(file_path, {
-                            recursive: true,
-                            force: true,
-                        });
-                    }
                 }catch(e){
                     // File was somehow deleted in the time that we read the dir to when we checked the file
-                    logger.warn(`Error removing file ${file_path}: ${e}`);
+                    logger.warn(`Error removing file ${file_path}: ${e}`)
                 }
             }
+
         }
 
         await fs.rm(this.dir, { recursive: true, force: true });
@@ -275,11 +262,12 @@ class Job {
         
         await Promise.all([
             this.cleanup_processes(),
-            this.cleanup_filesystem(),
+            this.cleanup_filesystem()
         ]);
     }
+
 }
 
 module.exports = {
-    Job,
+    Job
 };

api/src/package.js

@@ -13,6 +13,7 @@ const chownr = require('chownr');
 const util = require('util');
 
 class Package {
+
     constructor({ language, version, download, checksum }){
         this.language = language;
         this.version = semver.parse(version);
@@ -21,9 +22,7 @@ class Package {
     }
 
     get installed() {
-        return fss.exists_sync(
+        return fss.exists_sync(path.join(this.install_path, globals.pkg_installed_file));
-            path.join(this.install_path, globals.pkg_installed_file)
-        );
     }
 
     get install_path() {
@@ -43,18 +42,14 @@ class Package {
         logger.info(`Installing ${this.language}-${this.version.raw}`);
 
         if (fss.exists_sync(this.install_path)) {
-            logger.warn(
+            logger.warn(`${this.language}-${this.version.raw} has residual files. Removing them.`);
-                `${this.language}-${this.version.raw} has residual files. Removing them.`
-            );
             await fs.rm(this.install_path, { recursive: true, force: true });
         }
 
         logger.debug(`Making directory ${this.install_path}`);
         await fs.mkdir(this.install_path, {recursive: true});
 
-        logger.debug(
+        logger.debug(`Downloading package from ${this.download} in to ${this.install_path}`);
-            `Downloading package from ${this.download} in to ${this.install_path}`
-        );
         const pkgpath = path.join(this.install_path, 'pkg.tar.gz');
         const download = await fetch(this.download);
 
@@ -68,8 +63,7 @@ class Package {
 
         logger.debug('Validating checksums');
         logger.debug(`Assert sha256(pkg.tar.gz) == ${this.checksum}`);
-        const cs = crypto
+        const cs = crypto.create_hash("sha256")
-            .create_hash('sha256')
             .update(fss.readFileSync(pkgpath))
             .digest('hex');
 
@@ -77,14 +71,10 @@ class Package {
             throw new Error(`Checksum miss-match want: ${val} got: ${cs}`);
         }
 
-        logger.debug(
+        logger.debug(`Extracting package files from archive ${pkgpath} in to ${this.install_path}`);
-            `Extracting package files from archive ${pkgpath} in to ${this.install_path}`
-        );
 
         await new Promise((resolve, reject) => {
-            const proc = cp.exec(
+            const proc = cp.exec(`bash -c 'cd "${this.install_path}" && tar xzf ${pkgpath}'`);
-                `bash -c 'cd "${this.install_path}" && tar xzf ${pkgpath}'`
-            );
 
             proc.once('exit', (code, _) => {
                 code === 0 ? resolve() : reject();
@@ -105,11 +95,12 @@ class Package {
         const envout = await new Promise((resolve, reject) => {
             let stdout = '';
 
-            const proc = cp.spawn(
+            const proc = cp
+                .spawn(
                     'env',
                     ['-i','bash','-c',`${get_env_command}`],
                     {
-                    stdio: ['ignore', 'pipe', 'pipe'],
+                        stdio: ['ignore', 'pipe', 'pipe']
                     }
                 );
 
@@ -126,12 +117,7 @@ class Package {
 
         const filtered_env = envout
             .split('\n')
-            .filter(
+            .filter(l => !['PWD','OLDPWD','_', 'SHLVL'].includes(l.split('=',2)[0]))
-                l =>
-                    !['PWD', 'OLDPWD', '_', 'SHLVL'].includes(
-                        l.split('=', 2)[0]
-                    )
-            )
             .join('\n');
 
         await fs.write_file(path.join(this.install_path, '.env'), filtered_env);
@@ -140,55 +126,49 @@ class Package {
         await util.promisify(chownr)(this.install_path,0,0);
 
         logger.debug('Writing installed state to disk');
-        await fs.write_file(
+        await fs.write_file(path.join(this.install_path, globals.pkg_installed_file), Date.now().toString());
-            path.join(this.install_path, globals.pkg_installed_file),
-            Date.now().toString()
-        );
 
         logger.info(`Installed ${this.language}-${this.version.raw}`);
 
         return {
             language: this.language,
-            version: this.version.raw,
+            version: this.version.raw
         };
     }
 
+
     async uninstall(){
         logger.info(`Uninstalling ${this.language}-${this.version.raw}`);
 
-        logger.debug('Finding runtime');
+        logger.debug("Finding runtime")
-        const found_runtime = runtime.get_runtime_by_name_and_version(
+        const found_runtime = runtime.get_runtime_by_name_and_version(this.language, this.version.raw);
-            this.language,
-            this.version.raw
-        );
 
         if(!found_runtime){
-            logger.error(
+            logger.error(`Uninstalling ${this.language}-${this.version.raw} failed: Not installed`)
-                `Uninstalling ${this.language}-${this.version.raw} failed: Not installed`
+            throw new Error(`${this.language}-${this.version.raw} is not installed`)
-            );
-            throw new Error(
-                `${this.language}-${this.version.raw} is not installed`
-            );
         }
 
-        logger.debug('Unregistering runtime');
+        logger.debug("Unregistering runtime")
         found_runtime.unregister();
 
-        logger.debug('Cleaning files from disk');
+        logger.debug("Cleaning files from disk")
-        await fs.rmdir(this.install_path, { recursive: true });
+        await fs.rmdir(this.install_path, {recursive: true})
 
-        logger.info(`Uninstalled ${this.language}-${this.version.raw}`);
+        logger.info(`Uninstalled ${this.language}-${this.version.raw}`)
 
         return {
             language: this.language,
-            version: this.version.raw,
+            version: this.version.raw
         };
+
     }
 
     static async get_package_list() {
         const repo_content = await fetch(config.repo_url).then(x => x.text());
     
-        const entries = repo_content.split('\n').filter(x => x.length > 0);
+        const entries = repo_content
+            .split('\n')
+            .filter(x => x.length > 0);
     
         return entries.map(line => {
             const [ language, version, checksum, download ] = line.split(',', 4);
@@ -197,7 +177,7 @@ class Package {
                 language,
                 version,
                 checksum,
-                download,
+                download
             });
         });
     }
@@ -205,16 +185,16 @@ class Package {
     static async get_package (lang, version) {
         const packages = await Package.get_package_list();
     
-        const candidates = packages.filter(pkg => {
+        const candidates = packages
-            return (
+            .filter(pkg => {
-                pkg.language == lang && semver.satisfies(pkg.version, version)
+                return pkg.language == lang && semver.satisfies(pkg.version, version)
-            );
             });
     
         candidates.sort((a, b) => semver.rcompare(a.version, b.version));
     
         return candidates[0] || null;
     }
+
 }
 
 module.exports = Package;

api/src/runtime.js

@@ -8,6 +8,7 @@ const path = require('path');
 const runtimes = [];
 
 class Runtime {
+
     constructor({language, version, aliases, pkgdir, runtime}){
         this.language = language;
         this.version = version;
@@ -34,28 +35,26 @@ class Runtime {
         if(provides){
             // Multiple languages in 1 package
             provides.forEach(lang => {
-                runtimes.push(
+                runtimes.push(new Runtime({
-                    new Runtime({
                     language: lang.language,
                     aliases: lang.aliases,
                     version,
                     pkgdir: package_dir,
-                        runtime: language,
+                    runtime: language
-                    })
+                }));
-                );
             });
         }else{
-            runtimes.push(
+            runtimes.push(new Runtime({
-                new Runtime({
                 language,
                 version,
                 aliases,
-                    pkgdir: package_dir,
+                pkgdir: package_dir
-                })
+            }))
-            );
         }
 
         logger.debug(`Package ${language}-${version} was loaded`);
+
+        
     }
 
     get compiled() {
@@ -98,28 +97,15 @@ class Runtime {
 module.exports = runtimes;
 module.exports.Runtime = Runtime;
 module.exports.get_runtimes_matching_language_version = function(lang, ver){
-    return runtimes.filter(
+    return runtimes.filter(rt => (rt.language == lang || rt.aliases.includes(lang)) && semver.satisfies(rt.version, ver));
-        rt =>
-            (rt.language == lang || rt.aliases.includes(lang)) &&
-            semver.satisfies(rt.version, ver)
-    );
 };
-module.exports.get_latest_runtime_matching_language_version = function (
+module.exports.get_latest_runtime_matching_language_version = function(lang, ver){
-    lang,
+    return module.exports.get_runtimes_matching_language_version(lang, ver)
-    ver
-) {
-    return module.exports
-        .get_runtimes_matching_language_version(lang, ver)
         .sort((a,b) => semver.rcompare(a.version, b.version))[0];
 };
 
 module.exports.get_runtime_by_name_and_version = function(runtime, ver){
-    return runtimes.find(
+    return runtimes.find(rt => (rt.runtime == runtime || (rt.runtime === undefined && rt.language == runtime)) && semver.satisfies(rt.version, ver));
-        rt =>
+}
-            (rt.runtime == runtime ||
-                (rt.runtime === undefined && rt.language == runtime)) &&
-            semver.satisfies(rt.version, ver)
-    );
-};
 
 module.exports.load_package = Runtime.load_package;

cli/commands/execute.js

@@ -8,7 +8,7 @@ exports.aliases = ['run'];
 exports.describe = 'Executes file with the specified runner';
 
 exports.builder = {
-    language_version: {
+    languageVersion: {
         string: true,
         desc: 'Set the version of the language to use',
         alias: ['l'],
@@ -38,7 +38,7 @@ exports.builder = {
     }
 };
 
-exports.handler = async (argv) => {
+exports.handler = async function(argv) {
     const files = [...(argv.files || []),argv.file]
         .map(file_path => {
             return {
@@ -55,7 +55,7 @@ exports.handler = async (argv) => {
 
     const request = {
         language: argv.language,
-        version: argv['language_version'],
+        version: argv['language-version'],
         files: files,
         args: argv.args,
         stdin,
@@ -63,7 +63,7 @@ exports.handler = async (argv) => {
         run_timeout: argv.rt
     };
 
-    let { data: response } = await argv.axios.post('/api/v2/execute', request);
+    let { data: response } = await argv.axios.post('/api/v1/execute', request);
 
     const step = (name, ctx) => {
         console.log(chalk.bold(`== ${name} ==`));

cli/commands/ppman.js

@@ -1,7 +1,7 @@
-exports.command = 'ppman';
+exports.command = 'ppman'
-exports.aliases = ['pkg'];
+exports.aliases = ['pkg']
-exports.describe = 'Package Manager';
+exports.describe = 'Package Manager'
 
 exports.builder = yargs => yargs
     .commandDir('ppman_commands')
-    .demandCommand();
+    .demandCommand()

cli/commands/ppman_commands/install.js

@@ -1,23 +1,20 @@
 const chalk = require('chalk');
 
-exports.command = ['install <language> [language_version]'];
+exports.command = ['install <language> [language-version]']
-exports.aliases = ['i'];
+exports.aliases = ['i']
-exports.describe = 'Installs the named package';
+exports.describe = 'Installs the named package'
+
 
 const msg_format = {
-    color: p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
+    'color': p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Installation ${p.language ? "succeeded" : "failed: " + p.message}`,
-    monochrome: p => `Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
+    'monochrome': p => `Installation ${p.language ? "succeeded" : "failed: " + p.message}`,
-    json: JSON.stringify
+    'json': JSON.stringify
-};
 
-exports.handler = async ({ axios, language, language_version }) => {
+}
+
+exports.handler = async function({axios, language, languageVersion}){
     try{
-        const request = {
+        const install = await axios.post(`/api/v1/packages/${language}/${languageVersion || '*'}`)
-            language,
-            version: language_version || '*'
-        };
-
-        const install = await axios.post(`/api/v2/packages`, request);
 
         console.log(msg_format.color(install.data));
     }catch({response}){

cli/commands/ppman_commands/list.js

@@ -1,17 +1,22 @@
+//const fetch = require('node-fetch');
 const chalk = require('chalk');
 
-exports.command = ['list'];
+exports.command = ['list']
-exports.aliases = ['l'];
+exports.aliases = ['l']
-exports.describe = 'Lists all available packages';
+exports.describe = 'Lists all available packages'
+
 
 const msg_format = {
-    color: p => `${chalk[p.installed ? 'green':'red']('•')} ${p.language} ${p.language_version}`,
+    'color': p => `${chalk[p.installed ? "green":"red"]("•")} ${p.language} ${p.language_version}`,
-    monochrome: p => `${p.language} ${p.language_version} ${p.installed ? '(INSTALLED)': ''}`,
+    'monochrome': p => `${p.language} ${p.language_version} ${p.installed ? "(INSTALLED)": ""}`,
-    json: JSON.stringify
+    'json': JSON.stringify
-};
+
+}
+
+exports.handler = async function({axios}){
+
+    const packages = await axios.get('/api/v1/packages');
 
-exports.handler = async ({ axios }) => {
-    const packages = await axios.get('/api/v2/packages');
 
     const pkg_msg = packages.data
         .map(msg_format.color)

cli/commands/ppman_commands/uninstall.js

@@ -1,22 +1,20 @@
 const chalk = require('chalk');
 
-exports.command = ['uninstall <language> [language_version]'];
+exports.command = ['uninstall <language> [language-version]']
-exports.aliases = ['u'];
+exports.aliases = ['u']
-exports.describe = 'Uninstalls the named package';
+exports.describe = 'Uninstalls the named package'
+
 
 const msg_format = {
-    color: p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
+    'color': p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Uninstallation ${p.language ? "succeeded" : "failed: " + p.message}`,
-    monochrome: p => `Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
+    'monochrome': p => `Uninstallation ${p.language ? "succeeded" : "failed: " + p.message}`,
-    json: JSON.stringify
+    'json': JSON.stringify
-};
 
-exports.handler = async ({ axios, language, language_version }) => {
+}
+
+exports.handler = async function({axios, language, languageVersion}){
     try{
-        const request = {
+        const uninstall = await axios.delete(`/api/v1/packages/${language}/${languageVersion || '*'}`)
-            language,
-            version: language_version || '*'
-        };
-        const uninstall = await axios.delete(`/api/v2/packages`, {data: request});
 
         console.log(msg_format.color(uninstall.data));
     }catch({response}){

cli/index.js

@@ -2,12 +2,9 @@
 
 const axios = require('axios').default;
 
-const axios_instance = argv => {
+const axios_instance = function(argv){
     argv.axios = axios.create({
-        baseURL: argv['piston-url'],
+        baseURL: argv['piston-url']
-        headers: {
-            'Content-Type': 'application/json'
-        }
     });
 
     return argv;
@@ -21,7 +18,7 @@ require('yargs')(process.argv.slice(2))
         string: true
     })
     .middleware(axios_instance)
-    .scriptName('piston')
+    .scriptName("piston")
     .commandDir('commands')
     .demandCommand()
     .help()