Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Why don't we stop unix_server from listening in shutdown stage as well? #6393

Closed
Elliot438b opened this issue Nov 27, 2018 · 7 comments
Closed

Comments

@Elliot438b
Copy link
Contributor

my->https_server.stop_listening();

add


      if(my->unix_server.is_listening())
         my->unix_server.stop_listening();
@Elliot438b Elliot438b changed the title why we don't stop unix_server from listening in shutdown stage as well? Why don't we stop unix_server from listening in shutdown stage as well? Nov 27, 2018
@jgiszczak
Copy link
Contributor

jgiszczak commented Nov 27, 2018

We should also unlink the socket file. Pull requests gratefully accepted. Please branch from the develop branch.

@spoonincode
Copy link
Contributor

We should also unlink the socket file

Already does that

@Elliot438b
Copy link
Contributor Author

@spoonincode Can you kindly show me the position of source code?

@spoonincode
Copy link
Contributor

::unlink(m_acceptor->local_endpoint().path().c_str());

@spoonincode
Copy link
Contributor

oh and here too, so it looks like stop_listenting() should remove it

::unlink(m_acceptor->local_endpoint().path().c_str());

@Elliot438b
Copy link
Contributor Author

Elliot438b commented Nov 28, 2018

Yes, I think the destructor is not enough. The shutdown stage needs explicitly call.

@spoonincode
Copy link
Contributor

Fix merged

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants