Skip to content

Commit 10183d0

Browse files
authored
Merge pull request #494 from Brikaa/runtime-query-no-v3
Specify runtime in execution request, get rid of v3 middleware
2 parents 09eb9e4 + 605bef6 commit 10183d0

File tree

3 files changed

+8
-293
lines changed

3 files changed

+8
-293
lines changed

api/src/api/v2.js

+8-2
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ function get_job(job_info, available_runtimes) {
5151
let {
5252
language,
5353
version,
54+
runtime,
5455
args,
5556
stdin,
5657
files,
@@ -80,15 +81,20 @@ function get_job(job_info, available_runtimes) {
8081
}
8182
}
8283

84+
const has_runtime =
85+
job_info.has_own_property('runtime') && job_info.runtime !== null;
86+
8387
const rt = available_runtimes.find(
8488
rt =>
8589
[...rt.aliases, rt.language].includes(language) &&
86-
(version === rt.version || version === '*')
90+
(version === rt.version || version === '*') &&
91+
(!has_runtime || runtime === rt.runtime)
8792
);
8893

8994
if (rt === undefined) {
95+
const runtime_str = has_runtime ? `${runtime}-` : '';
9096
return reject({
91-
message: `${language}-${version} runtime is unknown`,
97+
message: `${runtime_str}${language}-${version} runtime is unknown`,
9298
});
9399
}
94100

api/src/api/v3.js

-289
This file was deleted.

api/src/bin/pistond.js

-2
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,7 @@ expressWs(app);
8787
logger.debug('Registering Routes');
8888

8989
const api_v2 = require('../api/v2');
90-
const api_v3 = require('../api/v3');
9190
app.use('/api/v2', api_v2);
92-
app.use('/api/v3', api_v3);
9391

9492
app.use((req, res, next) => {
9593
return res.status(404).send({ message: 'Not Found' });

0 commit comments

Comments
 (0)