Manage Tomcat queued connections

Adding two tomcat server properties:
- server.tomcat.accept-count
- server.tomcat.max-connections

Fixes gh-6433

Author: herau

spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/ServerProperties.java

@@ -87,6 +87,7 @@
  * @author Eddú Meléndez
  * @author Quinten De Swaef
  * @author Venil Noronha
+ * @author Aurélien Leboulanger
  */
 @ConfigurationProperties(prefix = "server", ignoreUnknownFields = true)
 public class ServerProperties
@@ -643,7 +644,7 @@ public static class Tomcat {
 
 		/**
 		 * Maximum size in bytes of the HTTP message header.
-		 */
+		*/
 		private int maxHttpHeaderSize = 0; // bytes
 
 		/**
@@ -657,6 +658,19 @@ public static class Tomcat {
 		 */
 		private Charset uriEncoding;
 
+		/**
+		 * Maximum amount of connections accept and process.
+		 * <p>Once the limit has been reached,
+		 * the operating system may still accept connections based on the @link{acceptCount} setting.</p>
+		 */
+		private int maxConnections = 0;
+
+		/**
+		 * Maximum queue length for incoming connection requests when all possible request processing threads are in use.
+		 * Any requests received when the queue is full will be refused. The default value is 100.
+		 */
+		private int acceptCount = 100;
+
 		public int getMaxThreads() {
 			return this.maxThreads;
 		}
@@ -772,6 +786,22 @@ public void setUriEncoding(Charset uriEncoding) {
 			this.uriEncoding = uriEncoding;
 		}
 
+		public int getMaxConnections() {
+			return this.maxConnections;
+		}
+
+		public void setMaxConnections(int maxConnections) {
+			this.maxConnections = maxConnections;
+		}
+
+		public int getAcceptCount() {
+			return this.acceptCount;
+		}
+
+		public void setAcceptCount(int acceptCount) {
+			this.acceptCount = acceptCount;
+		}
+
 		void customizeTomcat(ServerProperties serverProperties,
 				TomcatEmbeddedServletContainerFactory factory) {
 			if (getBasedir() != null) {
@@ -799,24 +829,40 @@ void customizeTomcat(ServerProperties serverProperties,
 			if (getUriEncoding() != null) {
 				factory.setUriEncoding(getUriEncoding());
 			}
-			if (serverProperties.getConnectionTimeout() != null) {
-				customizeConnectionTimeout(factory,
-						serverProperties.getConnectionTimeout());
+			if (this.maxConnections > 0) {
+				customizeMaxConnections(factory);
 			}
-			if (this.redirectContextRoot != null) {
-				customizeRedirectContextRoot(factory, this.redirectContextRoot);
+			if (this.acceptCount != 100) {
+				customizeAcceptCount(factory);
 			}
 		}
 
-		private void customizeConnectionTimeout(
-				TomcatEmbeddedServletContainerFactory factory, int connectionTimeout) {
-			for (Connector connector : factory.getAdditionalTomcatConnectors()) {
-				if (connector.getProtocolHandler() instanceof AbstractProtocol) {
-					AbstractProtocol<?> handler = (AbstractProtocol<?>) connector
-							.getProtocolHandler();
-					handler.setConnectionTimeout(connectionTimeout);
+		private void customizeAcceptCount(TomcatEmbeddedServletContainerFactory factory) {
+			factory.addConnectorCustomizers(new TomcatConnectorCustomizer() {
+
+				@Override
+				public void customize(Connector connector) {
+					ProtocolHandler handler = connector.getProtocolHandler();
+					if (handler instanceof AbstractProtocol) {
+						AbstractProtocol protocol = (AbstractProtocol) handler;
+						protocol.setBacklog(Tomcat.this.acceptCount);
+					}
 				}
-			}
+			});
+		}
+
+		private void customizeMaxConnections(TomcatEmbeddedServletContainerFactory factory) {
+			factory.addConnectorCustomizers(new TomcatConnectorCustomizer() {
+
+				@Override
+				public void customize(Connector connector) {
+					ProtocolHandler handler = connector.getProtocolHandler();
+					if (handler instanceof AbstractProtocol) {
+						AbstractProtocol protocol = (AbstractProtocol) handler;
+						protocol.setMaxConnections(Tomcat.this.maxConnections);
+					}
+				}
+			});
 		}
 
 		private void customizeBackgroundProcessorDelay(

spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/ServerPropertiesTests.java

@@ -321,6 +321,22 @@ public void testCustomizeTomcatMinSpareThreads() throws Exception {
 		assertThat(this.properties.getTomcat().getMinSpareThreads()).isEqualTo(10);
 	}
 
+	@Test
+	public void testCustomizeTomcatAcceptCount() throws Exception {
+		Map<String, String> map = new HashMap<String, String>();
+		map.put("server.tomcat.accept-count", "10");
+		bindProperties(map);
+		assertThat(this.properties.getTomcat().getAcceptCount()).isEqualTo(10);
+	}
+
+	@Test
+	public void testCustomizeTomcatMaxConnections() throws Exception {
+		Map<String, String> map = new HashMap<String, String>();
+		map.put("server.tomcat.max-connections", "5");
+		bindProperties(map);
+		assertThat(this.properties.getTomcat().getMaxConnections()).isEqualTo(5);
+	}
+
 	@Test
 	public void customizeTomcatDisplayName() throws Exception {
 		Map<String, String> map = new HashMap<String, String>();