The server does not close after closing the application (win.close) about electron-with-express HOT 15 CLOSED

Sorry for talking about something not related, but this might be a good time to add an issue template with steps to reproduce and context to approach problems better.

from electron-with-express.

Yep thank you so much @vzqzac for the pull request. I've merged it.

from electron-with-express.

The thing is, is that the BrowserWindow is a different process than the server so while the window will close it won't automatically close the server. Have a look at this issue: #11 as this method was adopted. Before you close the browser window send an IPC message 'stop-server' so that the server will close.

That said, this method is a stop gap measure, we can do better than this and we really should. If you play around with the code and find a better way to do it please send a pull request.

from electron-with-express.

yes okay but there is already an event

on("close", ...)

with

mainWindow.webContents.send('stop-server');

in main.js.
Then why when I do windoww.close(); does the server not close?

from electron-with-express.

Then why when I do windoww.close(); does the server not close?

All I can say is that the event may not be firing, or the IPC message isn't working so the server isn't being closed. It used to work. When I get some time I'll take a look.

from electron-with-express.

What platform are you on? I have been looking at this today and I cannot reproduce the issue on Windows 10.

from electron-with-express.

I'm running from command line using npm start and then I close the Electron window and the processes are all closing as expected. They are all closing even without the extra bit of code to tell the server to quit. So I think Windows is taking care of it but this may be an issue on Linux or Mac but I'm not sure.

from electron-with-express.

are you on a platform other than Windows?

from electron-with-express.

Sorry I don't think to check my emails... Yes, I'm on Linux. But the server does not close only when I close the application with win.close()

from electron-with-express.

It could be that the Window close event that is standard bypasses the Electron BrowserWindow close event. That's interesting and I'd think it'd be a bug in Electron if that were the case. I'll have to try on a Linux VM to see how it behaves. Give me some time because I just started a new job this morning.

from electron-with-express.

Totally agree @vzqzac !!!! If you have some ideas for this template please let me know and I'll gladly incorporate those ideas into it =)

Thank you for the suggestion!!!

from electron-with-express.

@vzqzac I don't even know how to create a issue template. Is this something that is added to the code repo and then is used by the issue tracker? Can you show me how to do it or point me to a reference on how to do it?

from electron-with-express.

Thanks for your response @frankhale, I just opened #16 with what I think is the proper information to add the templates, I'd appreciate your review there and of course any change request is highly appreciated too!

from electron-with-express.

So as I rewrite this I am finding that this is a really big problem. Sometimes the processes will exit and other times they won't. This is super annoying. I wonder if it's any better on Linux or Mac. Windows is a serious pain in the @$$!

from electron-with-express.

Closing this in favor of tracking this in #21

from electron-with-express.