Mascali
/
piston
mirror of https://github.com/engineer-man/piston.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

lint like rest of codebase

This commit is contained in:
Thomas Hobson 2021-05-08 12:30:40 +12:00
parent 1b7504a191
commit b3be57e0b4
No known key found for this signature in database
GPG Key ID: 9F1FD9D87950DB6F
10 changed files with 1893 additions and 1864 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
api/.prettierrc.yaml
View File

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

4
api/src/api/v2.js
View File

@ -136,7 +136,7 @@ router.post('/execute', async (req, res) => {
});
router.get('/runtimes', (req, res) => {
const runtimes = runtime.map((rt) => {
const runtimes = runtime.map(rt => {
return {
language: rt.language,
version: rt.version.raw,
@ -152,7 +152,7 @@ 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,

24
api/src/config.js
View File

@ -9,7 +9,7 @@ const options = [
default: 'INFO',
options: Object.values(Logger.LogLevels),
validators: [
(x) =>
x =>
Object.values(Logger.LogLevels).includes(x) ||
`Log level ${x} does not exist`,
],
@ -24,7 +24,9 @@ const options = [
key: 'data_directory',
desc: 'Absolute path to store all piston related data at',
default: '/piston',
validators: [(x) => fss.exists_sync(x) || `Directory ${x} does not exist`],
validators: [
x => fss.exists_sync(x) || `Directory ${x} does not exist`,
],
},
{
key: 'runner_uid_min',
@ -58,8 +60,8 @@ const options = [
key: 'disable_networking',
desc: 'Set to true to disable networking',
default: true,
parser: (x) => x === 'true',
validators: [(x) => typeof x === 'boolean' || `${x} is not a boolean`],
parser: x => x === 'true',
validators: [x => typeof x === 'boolean' || `${x} is not a boolean`],
},
{
key: 'output_max_size',
@ -99,7 +101,8 @@ const options = [
},
{
key: 'run_memory_limit',
desc: 'Max memory usage for run stage in bytes (set to -1 for no limit)',
desc:
'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`],
@ -119,10 +122,10 @@ let errored = false;
let config = {};
options.forEach((option) => {
options.forEach(option => {
const env_key = 'PISTON_' + option.key.to_upper_case();
const parser = option.parser || ((x) => x);
const parser = option.parser || (x => x);
const env_val = process.env[env_key];
@ -130,14 +133,17 @@ options.forEach((option) => {
const value = env_val || option.default;
option.validators.for_each((validator) => {
option.validators.for_each(validator => {
let response = null;
if (env_val) response = validator(parsed_val, env_val);
else response = validator(value, value);
if (response !== true) {
errored = true;
logger.error(`Config option ${option.key} failed validation:`, response);
logger.error(
`Config option ${option.key} failed validation:`,
response
);
return;
}
});

2
api/src/globals.js
View File

@ -5,7 +5,7 @@ const platform = `${is_docker() ? 'docker' : 'baremetal'}-${fs
.read_file_sync('/etc/os-release')
.toString()
.split('\n')
.find((x) => x.startsWith('ID'))
.find(x => x.startsWith('ID'))
.replace('ID=', '')}`;
module.exports = {

12
api/src/index.js
View File

@ -18,7 +18,7 @@ const app = express();
Logger.setLogLevel(config.log_level);
logger.debug('Ensuring data directories exist');
Object.values(globals.data_directories).for_each((dir) => {
Object.values(globals.data_directories).for_each(dir => {
let data_path = path.join(config.data_directory, dir);
logger.debug(`Ensuring ${data_path} exists`);
@ -43,20 +43,20 @@ const app = express();
const pkglist = await fs.readdir(pkgdir);
const languages = await Promise.all(
pkglist.map((lang) => {
return fs.readdir(path.join(pkgdir, lang)).then((x) => {
return x.map((y) => path.join(pkgdir, lang, y));
pkglist.map(lang => {
return fs.readdir(path.join(pkgdir, lang)).then(x => {
return 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))
);
installed_languages.for_each((pkg) => runtime.load_package(pkg));
installed_languages.for_each(pkg => runtime.load_package(pkg));
logger.info('Starting API Server');
logger.debug('Constructing Express App');

32
api/src/job.js
View File

@ -106,9 +106,12 @@ class Job {
proc.stdin.end();
proc.stdin.destroy();
const kill_timeout = set_timeout((_) => proc.kill('SIGKILL'), timeout);
const kill_timeout = set_timeout(
_ => proc.kill('SIGKILL'),
timeout
);
proc.stderr.on('data', (data) => {
proc.stderr.on('data', data => {
if (stderr.length > config.output_max_size) {
proc.kill('SIGKILL');
} else {
@ -117,7 +120,7 @@ class Job {
}
});
proc.stdout.on('data', (data) => {
proc.stdout.on('data', data => {
if (stdout.length > config.output_max_size) {
proc.kill('SIGKILL');
} else {
@ -139,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 });
@ -150,7 +153,8 @@ class Job {
async execute() {
if (this.state !== job_states.PRIMED) {
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()
);
}
@ -167,7 +171,7 @@ class Job {
if (this.runtime.compiled) {
compile = await this.safe_call(
path.join(this.runtime.pkgdir, 'compile'),
this.files.map((x) => x.name),
this.files.map(x => x.name),
this.timeouts.compile,
this.memory_limits.compile
);
@ -200,11 +204,11 @@ class Job {
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) => {
const procs = lines.map(line => {
const [pid, ruid] = line
.trim()
.split(/\s+/)
.map((n) => parseInt(n));
.map(n => parseInt(n));
return { pid, ruid };
});
@ -216,7 +220,7 @@ class Job {
})
);
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
@ -251,7 +255,10 @@ class Job {
const stat = await fs.stat(file_path);
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
@ -266,7 +273,10 @@ class Job {
async cleanup() {
logger.info(`Cleaning up job uuid=${this.uuid}`);
await Promise.all([this.cleanup_processes(), this.cleanup_filesystem()]);
await Promise.all([
this.cleanup_processes(),
this.cleanup_filesystem(),
]);
}
}

31
api/src/package.js
View File

@ -105,15 +105,19 @@ class Package {
const envout = await new Promise((resolve, reject) => {
let stdout = '';
const proc = cp.spawn('env', ['-i', 'bash', '-c', `${get_env_command}`], {
const proc = cp.spawn(
'env',
['-i', 'bash', '-c', `${get_env_command}`],
{
stdio: ['ignore', 'pipe', 'pipe'],
});
}
);
proc.once('exit', (code, _) => {
code === 0 ? resolve(stdout) : reject();
});
proc.stdout.on('data', (data) => {
proc.stdout.on('data', data => {
stdout += data;
});
@ -123,7 +127,10 @@ class Package {
const filtered_env = envout
.split('\n')
.filter(
(l) => !['PWD', 'OLDPWD', '_', 'SHLVL'].includes(l.split('=', 2)[0])
l =>
!['PWD', 'OLDPWD', '_', 'SHLVL'].includes(
l.split('=', 2)[0]
)
)
.join('\n');
@ -159,7 +166,9 @@ class Package {
logger.error(
`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');
@ -177,11 +186,11 @@ class Package {
}
static async get_package_list() {
const repo_content = await fetch(config.repo_url).then((x) => x.text());
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) => {
return entries.map(line => {
const [language, version, checksum, download] = line.split(',', 4);
return new Package({
@ -196,8 +205,10 @@ class Package {
static async get_package(lang, version) {
const packages = await Package.get_package_list();
const candidates = packages.filter((pkg) => {
return pkg.language == lang && semver.satisfies(pkg.version, version);
const candidates = packages.filter(pkg => {
return (
pkg.language == lang && semver.satisfies(pkg.version, version)
);
});
candidates.sort((a, b) => semver.rcompare(a.version, b.version));

8
api/src/runtime.js
View File

@ -33,7 +33,7 @@ class Runtime {
if (provides) {
// Multiple languages in 1 package
provides.forEach((lang) => {
provides.forEach(lang => {
runtimes.push(
new Runtime({
language: lang.language,
@ -76,7 +76,7 @@ class Runtime {
env_content
.trim()
.split('\n')
.map((line) => line.split('=', 2))
.map(line => line.split('=', 2))
.forEach(([key, val]) => {
this._env_vars[key.trim()] = val.trim();
});
@ -99,7 +99,7 @@ module.exports = runtimes;
module.exports.Runtime = Runtime;
module.exports.get_runtimes_matching_language_version = function (lang, ver) {
return runtimes.filter(
(rt) =>
rt =>
(rt.language == lang || rt.aliases.includes(lang)) &&
semver.satisfies(rt.version, ver)
);
@ -115,7 +115,7 @@ module.exports.get_latest_runtime_matching_language_version = function (
module.exports.get_runtime_by_name_and_version = function (runtime, ver) {
return runtimes.find(
(rt) =>
rt =>
(rt.runtime == runtime ||
(rt.runtime === undefined && rt.language == runtime)) &&
semver.satisfies(rt.version, ver)