Closed epoll server socket channel still accepting connections.

[Luke Daley]

Hi,

Does anyone know of any issues with epoll server socket channels continuing to accept incoming requests after channel close? The same code refuses new connections when not using the epoll transport.

I’m issuing channel.close().sync(), yet new connections are still being accepted.

I’m using 4.1.0.CR7. 

[Norman Maurer]

Nope… Could you please open an issue and as best also include a reproducer ?

Thanks Luke!

-- 
You received this message because you are subscribed to the Google Groups "Netty discussions" group.
To unsubscribe from this group and stop receiving emails from it, send an email to netty+un...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/netty/CAKn7%3D%2B%3DB6_eAeayY6M6e4DLCcG%2BSvyLOaDni6GSopVE_6B1L6Q%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

[Norman Maurer]

Just tested it with 4.1 branch and it works as expected. Maybe try upgrade and see if it fix it ?

[Luke Daley]

The same happens with 4.1.1.Final. 

[Norman Maurer]

 I was not able to to reproduce it. Can you procide a reproducer?

--

You received this message because you are subscribed to the Google Groups "Netty discussions" group.
To unsubscribe from this group and stop receiving emails from it, send an email to netty+un...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/netty/CAKn7%3D%2BkOGCYcYgU4VRUza_-9kLexDqWDuEVKeWQMGmR%3DhhhQdg%40mail.gmail.com.

[Luke Daley]

I've finally worked out what was going on here. It turns out that it was, unsurprisingly, a problem with my code. 

I had a test which effectively started then stopped a HTTP server app, then asserted that it eventually stopped accepting requests. This was done in parallel. There was an unlikely race in the test that meant a request may have made it before the shutdown was issued. This, combined with the fact that I didn't realise that the HTTP client I was using to test supported keep alive was the problem. The app was indeed rejecting new connections, but the requesting loop was reusing the open connection each time.

Cheers.

[Norman Maurer]

Thanks for closing the loop

To view this discussion on the web visit https://groups.google.com/d/msgid/netty/e6fa061c-92bc-41ab-8987-ccf0586e203c%40googlegroups.com.