Skip to content

Commit

Permalink
fix(cmd-api-server): disallow running on older than Node 12 but provi…
Browse files Browse the repository at this point in the history
…de optional override

1. Node 10 is no longer the LTS version
2. Node 10 does not properly support TLS v1.3
3. The CI script will no longer run against Node 10 just 12 and 14.

fix #150

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
  • Loading branch information
petermetz authored and sfuji822 committed Jun 30, 2020
1 parent 9dd28da commit 332b306
Show file tree
Hide file tree
Showing 5 changed files with 54 additions and 5 deletions.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
language: node_js
node_js:
- 10
- 12
- 14

Expand Down
14 changes: 14 additions & 0 deletions packages/cactus-cmd-api-server/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 2 additions & 3 deletions packages/cactus-cmd-api-server/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,10 @@
"types": "dist/types/main/typescript/index.d.ts",
"scripts": {
"tsc": "tsc --project ./tsconfig.json",

"webpack": "npm-run-all webpack:dev webpack:prod",

"webpack:dev": "npm-run-all webpack:dev:node webpack:dev:web",
"webpack:dev:web": "webpack --env=dev --target=web --config ../../webpack.config.js",
"webpack:dev:node": "webpack --env=dev --target=node --config ../../webpack.config.js",

"webpack:prod": "npm-run-all webpack:prod:node webpack:prod:web",
"webpack:prod:web": "webpack --env=prod --target=web --config ../../webpack.config.js",
"webpack:prod:node": "webpack --env=prod --target=node --config ../../webpack.config.js"
Expand Down Expand Up @@ -82,6 +79,7 @@
"js-sha3": "0.8.0",
"node-fetch": "3.0.0-beta.4",
"secp256k1": "4.0.0",
"semver": "7.3.2",
"sha3": "2.1.2",
"typescript-optional": "2.0.1",
"uuid": "7.0.2"
Expand All @@ -94,6 +92,7 @@
"@types/joi": "14.3.4",
"@types/multer": "1.4.2",
"@types/secp256k1": "3.5.3",
"@types/semver": "7.3.1",
"@types/uuid": "7.0.2"
}
}
26 changes: 25 additions & 1 deletion packages/cactus-cmd-api-server/src/main/typescript/api-server.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import path from "path";
import { Server } from "http";
import { gte } from "semver";
import express, {
Express,
Request,
Expand All @@ -18,7 +19,10 @@ import {
IPluginWebService,
PluginRegistry,
} from "@hyperledger/cactus-core-api";
import { ICactusApiServerOptions as ICactusApiServerConfig } from "./config/config-service";
import {
ICactusApiServerOptions as ICactusApiServerConfig,
ConfigService,
} from "./config/config-service";
import { CACTUS_OPEN_API_JSON } from "./openapi-spec";
import { Logger, LoggerProvider } from "@hyperledger/cactus-common";
import { Servers } from "./common/servers";
Expand Down Expand Up @@ -49,6 +53,7 @@ export class ApiServer {
}

async start(): Promise<void> {
this.checkNodeVersion();
try {
await this.startCockpitFileServer();
await this.startApiServer();
Expand All @@ -59,6 +64,25 @@ export class ApiServer {
}
}

/**
* Verifies that the currently running NodeJS process is at least of a certain
* NodeJS version as specified by the configuration.
*
* @throws {Error} if the version contraint is not satisfied by the runtime.
*/
public checkNodeVersion(currentVersion: string = process.version): void {
if (gte(this.options.config.minNodeVersion, currentVersion)) {
const msg =
`ApiServer#checkNodeVersion() detected NodeJS ` +
`v${process.version} that is outdated as per the configuration. ` +
`If you must run on this NodeJS version you can override the minimum ` +
`acceptable version via config parameters of the API server. ` +
`Though doing so may lead to vulnerabilities in your deployment. ` +
`You've been warned.`;
throw new Error(msg);
}
}

public getHttpServerApi(): Server | null {
return this.httpServerApi;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ export interface ICactusApiServerOptions {
privateKey: string;
keychainSuffixPublicKey: string;
keychainSuffixPrivateKey: string;
minNodeVersion: string;
}

export class ConfigService {
Expand Down Expand Up @@ -107,6 +108,17 @@ export class ConfigService {
env: "LOG_LEVEL",
arg: "log-level",
},
minNodeVersion: {
doc:
"Determines the lower bound of NodeJS version that the API " +
"server will be willing to start on. Defaults to v12 because v10 " +
"does not support TLS v1.3. If you must run on Node 10, just set " +
"this configuration parameter to 10.0.0 for example.",
format: ConfigService.formatNonBlankString,
default: "12.0.0",
env: "MIN_NODE_VERSION",
arg: "min-node-version",
},
cockpitHost: {
doc:
"The host to bind the Cockpit webserver to. Secure default is: 127.0.0.1. Use 0.0.0.0 to bind for any host.",
Expand Down Expand Up @@ -272,6 +284,7 @@ export class ConfigService {
configFile: ".config.json",
cactusNodeId: uuidV4(),
logLevel: "debug",
minNodeVersion: (schema.minNodeVersion as SchemaObj).default,
publicKey,
privateKey,
apiCorsDomainCsv: (schema.apiCorsDomainCsv as SchemaObj).default,
Expand Down

0 comments on commit 332b306

Please sign in to comment.