Keep defaulting tty to true to avoid accidental breakage

Otherwise old releases (where applications expected tty to be true)
would break.

Change-type: patch
Signed-off-by: Pablo Carranza Velez <pablo@balena.io>
This commit is contained in:
Pablo Carranza Velez 2018-10-19 17:50:12 +02:00
parent f89af2d508
commit 32d5e58ead

View File

@ -258,7 +258,9 @@ export class Service {
// Sanity check the incoming boolean values
config.oomKillDisable = Boolean(config.oomKillDisable);
config.readOnly = Boolean(config.readOnly);
config.tty = Boolean(config.tty);
if (config.tty != null) {
config.tty = Boolean(config.tty);
}
if (_.isArray(config.sysctls)) {
config.sysctls = _.fromPairs(_.map(config.sysctls, (v) => _.split(v, '=')));
@ -343,7 +345,7 @@ export class Service {
hostname: '',
user: '',
workingDir: '',
tty: false,
tty: true,
});
// Mutate service with extra features