robojerk/codeql-action

match existing behaviour when custom listeners defined

Browse source
This commit is contained in:
Nick Fyson 2020-08-24 18:35:22 +01:00
parent 2b27c68c84
commit 45e00a8e6a
3 changed files with 30 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

18
src/exec_wrapper.ts
View file

@ -11,14 +11,22 @@ export async function exec_wrapper(commandLine: string, args?: string[], options
let listeners = {
stdout: (data: Buffer) => {
stdout += data.toString();
// NB change behaviour to only write to stdout/err if no listener passed
process.stdout.write(data);
originalListener?.stdout?.(data);
if (originalListener?.stdout !== undefined) {
originalListener.stdout(data);
} else {
// if no stdout listener was originally defined then match behaviour of exec.exec
process.stdout.write(data);
}
},
stderr: (data: Buffer) => {
stderr += data.toString();
process.stderr.write(data);
originalListener?.stderr?.(data);
if (originalListener?.stderr !== undefined) {
originalListener.stderr(data);
} else {
// if no stderr listener was originally defined then match behaviour of exec.exec
process.stderr.write(data);
}
}
};