Merge branch '3.2.x'

Author: wilkinsona

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/embedded/tomcat/GracefulShutdown.java

@@ -67,25 +67,15 @@ private void doShutdown(GracefulShutdownCallback callback, CountDownLatch shutdo
 			List<Connector> connectors = getConnectors();
 			connectors.forEach(this::close);
 			shutdownUnderway.countDown();
-			try {
-				for (Container host : this.tomcat.getEngine().findChildren()) {
-					for (Container context : host.findChildren()) {
-						while (!this.aborted && isActive(context)) {
-							Thread.sleep(50);
-						}
-						if (this.aborted) {
-							logger.info("Graceful shutdown aborted with one or more requests still active");
-							callback.shutdownComplete(GracefulShutdownResult.REQUESTS_ACTIVE);
-							return;
-						}
-					}
-				}
+			awaitInactiveOrAborted();
+			if (this.aborted) {
+				logger.info("Graceful shutdown aborted with one or more requests still active");
+				callback.shutdownComplete(GracefulShutdownResult.REQUESTS_ACTIVE);
 			}
-			catch (InterruptedException ex) {
-				Thread.currentThread().interrupt();
+			else {
+				logger.info("Graceful shutdown complete");
+				callback.shutdownComplete(GracefulShutdownResult.IDLE);
 			}
-			logger.info("Graceful shutdown complete");
-			callback.shutdownComplete(GracefulShutdownResult.IDLE);
 		}
 		finally {
 			shutdownUnderway.countDown();
@@ -105,6 +95,22 @@ private void close(Connector connector) {
 		connector.getProtocolHandler().closeServerSocketGraceful();
 	}
 
+	private void awaitInactiveOrAborted() {
+		try {
+			for (Container host : this.tomcat.getEngine().findChildren()) {
+				for (Container context : host.findChildren()) {
+					while (!this.aborted && isActive(context)) {
+						Thread.sleep(50);
+					}
+				}
+			}
+
+		}
+		catch (InterruptedException ex) {
+			Thread.currentThread().interrupt();
+		}
+	}
+
 	private boolean isActive(Container context) {
 		try {
 			if (((StandardContext) context).getInProgressAsyncCount() > 0) {