search

coreybutler / node-windows

Windows support for Node.JS scripts (daemons, eventlog, UAC, etc).
Other
2.78k stars 357 forks source link

Missing comma in lib/Wrapper.js #271

Closed marichard-incomm closed 3 years ago

marichard-incomm commented 3 years ago

There is a missing comma in wrapper.js, in line 69: child = null I think is breaking things in my environment, is that possible?

jvicente-incomm commented 3 years ago

Follow up on manuRichard's comment. This is the log of the error we see when deploying our system: C:\MY_DIR\node_modules\node-windows\lib\wrapper.js:177 child.send('shutdown'); ____^

TypeError: Cannot read property 'send' of null at process.killkid (C:\MY_DIR\node_modules\node-windows\lib\wrapper.js:177:11) at process.emit (events.js:321:20) at process.exit (internal/process/per_thread.js:165:15) at monitor (C:\MY_DIR\node_modules\node-windows\lib\wrapper.js:99:17) at ChildProcess. (C:\MY_DIR\node_modules\node-windows\lib\wrapper.js:170:5) at ChildProcess.emit (events.js:321:20) at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)

marichard-incomm commented 3 years ago

Closing this, sorry. The missing comma is not causing the issue I'm having.