Bug 50406 - init() doesn't initialize connector any more
Summary: init() doesn't initialize connector any more
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 7
Classification: Unclassified
Component: Catalina (show other bugs)
Version: trunk
Hardware: PC Linux
: P2 regression (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-03 03:33 UTC by Mladen Turk
Modified: 2010-12-03 13:09 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mladen Turk 2010-12-03 03:33:30 UTC
Recent change of moving connectors init() code to start()
makes Tomcat unusable under commons-daemon.
This is severe regression cause sockets are now bind in start
method when the privileges are already dropped.
Comment 1 Christopher Schultz 2010-12-03 10:40:44 UTC
How about adding a method:

public void initAndStart()
{
  init();
  start();
}

Commons-daemon can use that. Of course, this introduces an API incompatibility and users will have to have a minimum commons-daemon version to work properly with TC 7.0.x.

Or would it be better to revert the recent patch and create new methods with different semantics? Since TC 7 is not considered stable (yet), it seems okay to change the API like this. On the other hand, it impacts an external project.
Comment 2 Mark Thomas 2010-12-03 13:09:07 UTC
Breaking change reverted.