https://issues.apache.org/bugzilla/show_bug.cgi?id=48470

--- Comment #6 from Konstantin Kolinko <knst.koli...@gmail.com> 2010-01-11 
17:19:28 UTC ---
attachment 24827 patch looks good, though I have not tried to run it yet.

+1 to add "Socket unlock completed for:" debug message to AprEndpoint, like it
is done in JIoEndpoint.


Regarding s.setSoLinger(true, 0):
I see that NioEndpoint of TC6 and all endpoint implementations of TC7 use
 s.setSoLinger(getSocketProperties().getSoLingerOn(), ...)

The default value of soLingerOn is based on Constants.DEFAULT_CONNECTION_LINGER
constants (in o.a.coyote.http11 or in o.a.coyote.ajp) that is -1. Thus it will
be false.

I think that s.setSoLinger(true, 0) should be used in unlockAccept() for its
dummy connection in all implementations of endpoint. Though I have not tested
it.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to