Author: remm
Date: Mon Nov 14 17:11:30 2016
New Revision: 1769665

URL: http://svn.apache.org/viewvc?rev=1769665&view=rev
Log:
Restore the default connector creation, the most important part is to not 
create it on start. The only broken use is an app which doesn't call 
getConnector (and needs only port configuration).

Modified:
    tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java
    tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java?rev=1769665&r1=1769664&r2=1769665&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java Mon Nov 14 
17:11:30 2016
@@ -166,8 +166,8 @@ public class Tomcat {
     }
 
     /**
-     * Set the port for the default connector. Must
-     * be called before start().
+     * Set the port for the default connector. The default connector will
+     * only be created if getConnector is called.
      * @param port The port number
      */
     public void setPort(int port) {
@@ -403,7 +403,14 @@ public class Tomcat {
         if (service.findConnectors().length > 0) {
             return service.findConnectors()[0];
         }
-        return null;
+        // The same as in standard Tomcat configuration.
+        // This creates an APR HTTP connector if AprLifecycleListener has been
+        // configured (created) and Tomcat Native library is available.
+        // Otherwise it creates a NIO HTTP connector.
+        Connector connector = new Connector("HTTP/1.1");
+        connector.setPort(port);
+        service.addConnector(connector);
+        return connector;
     }
 
     public void setConnector(Connector connector) {

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1769665&r1=1769664&r2=1769665&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Mon Nov 14 17:11:30 2016
@@ -66,8 +66,8 @@
         embedding Tomcat. (markt)
       </fix>
       <update>
-        <bug>60368</bug>: Stop creating a default connector in embedded mode.
-        (remm)
+        <bug>60368</bug>: Stop creating a default connector on start in
+        embedded mode. (remm)
       </update>
     </changelog>
   </subsection>



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

Reply via email to