maxmustermann/pulumi
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Properly report closes due to signals

Browse source
This commit is contained in:
joeduffy 2017-09-05 12:01:55 -07:00
parent e3a6695399
commit 0a78ef0743
1 changed files with 22 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
sdk/nodejs/runtime/langhost.ts
View file

@ -103,20 +103,31 @@ function runRPC(call: any, callback: any): void {
proc.stderr.on("data", (data: string | Buffer) => { proc.stderr.on("data", (data: string | Buffer) => {
console.error(stripEOL(data)); console.error(stripEOL(data));
}); });
// If we got this far, make sure to communicate completion when the process terminates.
proc.on("close", (code: number, signal: string) => {
if (callback !== undefined) {
if (code !== 0) {
if (signal) {
resp.setError(`Program exited due to a signal: ${signal}`);
}
else {
resp.setError(`Program exited with non-zero exit code: ${code}`);
}
}
callback(undefined, resp);
callback = undefined;
}
});
} }
catch (err) { catch (err) {
resp.setError(err.message); if (callback !== undefined) {
callback(undefined, resp); resp.setError(err.message);
return; callback(undefined, resp);
} callback = undefined;
// If we got this far, make sure to communicate completion when the process terminates.
proc.on("close", (code: number) => {
if (code !== 0) {
resp.setError(`Program exited with non-zero exit code: ${code}`);
} }
callback(undefined, resp); }
});
} }
function stripEOL(data: string | Buffer): string { function stripEOL(data: string | Buffer): string {