When calling exit(), added call to phantom.kill()

[ryanone]

Calling exit() doesn't terminate the spawned process. So I added a call to phantom.kill() to do the trick.

Fixes #24

[AndrewRayCode]

oh thank god. that explains why phantom never dies for me.

[alexscheelmeyer]

This does not work for me. Normally all PhantomJS instances exit correctly for me, but if I include this they stop exiting. Which platform are you on?

[ryanone]

Thanks for responding, Alex.

I'm on RHEL6.

@delvarworld - How about you?

[ndrut]

CentOS 6(RHEL 6) here -- Same issue. But, I assumed it was because I haven't figured out how to add a timeout to the phantom object.

[sandinmyjoints]

I see this, too: Phantom never dies. On OS X, using 1.8.1.

[kaareal]

Also using 1.8.1 on osx

[eddloschi]

This line fixes phantom process not exiting for me

• Arch Linux Kernel 3.8.4
• PhantomJS 1.9.0
• Node.js 0.10.2

[kaareal]

I think one need to wait for the kill event to be send.
I resolved this by waiting a bit before exiting the script.

[potmo]

having the same problem on
osx 10.7.5
phantomjs 1.9.0
Node 0.8.10

The instances are draining a lot of cpu as well from what I can see in
top -o cpu

Its the bridge.js that has started them from what I can see with
ps aux | grep phantom

[potmo]

The fix doesn't work for me. I think I get a script error in phantom that crashes phantom silently. The phantom-process is never killed

[AlexGilleran]

Whew, just spent hours wondering why I was spawning off phantomjs zombie processes. Fix works for me - I can't modify the node-phantom code so I'm working around it by using:

ph.exit();
ph._phantom.kill('SIGTERM');

[thisispete]

shouldn't the right place to put this be line 141: ?

case 'phantomExited':
                            request(socket,[0,'exitAck']);
                            server.close();
                            io.set('client store expiration', 0);
                            cmds[cmdId].cb();
                            phantom.kill('SIGTERM');
                            delete cmds[cmdId];
                            break;