DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #14 from Martin Grotzke  2010-12-09 
19:24:22 EST ---
(In reply to comment #13)
> Re-factoring goof - fixed.

Great, thanx a lot for your work! Now I ran into another issue, will submit
this separately (#50448).

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

Mark Thomas  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

--- Comment #13 from Mark Thomas  2010-12-09 16:12:08 EST ---
Re-factoring goof - fixed.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

Sylvain Laurent  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |

--- Comment #12 from Sylvain Laurent  2010-12-09 
15:30:05 EST ---
I think your changes broke /manager/status , I get the following exception with
rev 1044114 :


SEVERE: Servlet.service() for servlet [Status] in context with path [/manager]
threw exception [javax.management.AttributeNotFoundException:  Cannot find
attribute maxTime for
org.apache.coyote.http11.http11protocol$http11connectionhand...@2377ab84] with
root cause
javax.management.AttributeNotFoundException:  Cannot find attribute maxTime for
org.apache.coyote.http11.http11protocol$http11connectionhand...@2377ab84
at
org.apache.tomcat.util.modeler.ManagedBean.getGetter(ManagedBean.java:494)
at
org.apache.tomcat.util.modeler.BaseModelMBean.getAttribute(BaseModelMBean.java:180)
at
com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.getAttribute(DefaultMBeanServerInterceptor.java:666)
at
com.sun.jmx.mbeanserver.JmxMBeanServer.getAttribute(JmxMBeanServer.java:638)
at
org.apache.catalina.manager.StatusTransformer.writeConnectorState(StatusTransformer.java:275)
at
org.apache.catalina.manager.StatusManagerServlet.doGet(StatusManagerServlet.java:276)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:623)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:724)
at
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:306)
at
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
at
org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:240)
at
org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:161)
at
org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:561)
at
org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:164)
at
org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:108)
at
org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:558)
at
org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
at
org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:379)
at
org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:243)
at
org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:179)
at
org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:157)
at
org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:283)
at
java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:680)

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

Mark Thomas  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

--- Comment #11 from Mark Thomas  2010-12-09 14:54:52 EST ---
And fixed again. You want to set the bindOnInit parameter to false to get the
behaviour you want.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #10 from Mark Thomas  2010-12-09 06:32:46 EST ---
Yep. Working on it (well the clean up prior to implementing the fix) now. A
bindOnInit param will be added to the connector that defaults to true. By
default the connector will bind/unbind on init()/destroy(). Set it to false and
the connector will bind/unbind on start)_/stop()

I could have hacked this in fairly quickly but the connector/protocol/endpoint
code is long overdue a clean-up and this issue was the trigger for getting that
done. Connector and protocol are cleaned up. Starting on the endpoints now.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-08 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #9 from Martin Grotzke  2010-12-08 
19:15:46 EST ---
D'oh. Is there another solution in sight?

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-03 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

Mark Thomas  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |

--- Comment #8 from Mark Thomas  2010-12-03 13:00:10 EST ---
The original fix is going to have to be reverted. It caused issues with jsvc
(bug 50406) since the privileged ports need to be bound before the user id
switches to the non-root user.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-02 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #7 from Martin Grotzke  2010-12-02 
03:11:32 EST ---
Great, thanx a lot for fixing this!

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #6 from Mark Thomas  2010-12-01 13:38:53 EST ---
It wasn't as much work as I feared. Pretty much all a simple copy and paste.

The fix has been applied to 7.0.x and will be included in 7.0.6 onwards.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-12-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #5 from Martin Grotzke  2010-12-01 
03:24:07 EST ---
Ok, great you're on it already!

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-11-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #4 from Mark Thomas  2010-11-30 17:52:07 EST ---
I'm working on this at the moment (although work and infrastructure stuff may
distract me for the next few days). I have already sorted out the
MapperListener, the ProtocolHandler and Endpoints are next but they will take a
little longer as all five of the connectors will probably need to be changed
together.

>From what I have looked at so far most of init() needs to move into start(),
most of destroy() into stop() and any JMX stuff needs to move to init() /
destroy().

The new Lifecycle code can't be used directly as it will create a dependency on
Catalina from the Connectors but it should be possible to re-use most of the
ideas and probably some of the code.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-11-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #3 from Martin Grotzke  2010-11-30 
16:24:43 EST ---
I debugged this and found the following path of execution:

Connector.stopInternal():
-> protocolHandler.stop() (Http11Protocol, stop() implemented in
AbstractHttp11Protocol)
   -> endpoint.stop() (JIoEndpoint)
  -> pause()
  -> unlockAccept()
  -> shutdownExecutor()

A problem here seems to be that JIoEndpoint.stop() does not close the
serverSocket (only destroy is doing this).

Therefore, if the implementation of AbstractHttp11Protocol.stop() is changed to
invoke
  endpoint.destroy();
instead of endpoint.stop() also fixes the issue.

Not sure if this is the correct solution.

How can we proceed with this?

Thanx && Cheers,
Martin

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-11-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #2 from Mark Thomas  2010-11-29 08:47:21 EST ---
Connector components can be started and stopped multiple times and setting
MUST_DESTROY will break that. Therefore, the patch can't be used in its current
form.

The Connectors and some of their associated components have non-standard
lifecycles and I suspect that a change to one of the associated components will
be required to fix this.

-- 
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



DO NOT REPLY [Bug 50360] Server socket still bound after Embedded.stop is invoked

2010-11-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360

--- Comment #1 from Martin Grotzke  2010-11-28 
20:20:10 EST ---
Created an attachment (id=26352)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26352)
Add setState(LifecycleState.MUST_DESTROY) to Connector.stopInternal (based on
7.0.4)

-- 
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