From 8c37ad7ac5034faed74cb53fd37b9865adfd56a5 Mon Sep 17 00:00:00 2001
From: Rossen Stoyanchev <rstoyanchev@pivotal.io>
Date: Tue, 13 Feb 2018 13:44:08 -0500
Subject: [PATCH] Startup/shutdown log messages for AbstractHttpServer

Issue: SPR-16494
---
 .../reactive/bootstrap/AbstractHttpServer.java  | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/spring-web/src/test/java/org/springframework/http/server/reactive/bootstrap/AbstractHttpServer.java b/spring-web/src/test/java/org/springframework/http/server/reactive/bootstrap/AbstractHttpServer.java
index bf7f24ede1..8563c3ac98 100644
--- a/spring-web/src/test/java/org/springframework/http/server/reactive/bootstrap/AbstractHttpServer.java
+++ b/spring-web/src/test/java/org/springframework/http/server/reactive/bootstrap/AbstractHttpServer.java
@@ -19,15 +19,21 @@ package org.springframework.http.server.reactive.bootstrap;
 import java.util.LinkedHashMap;
 import java.util.Map;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.springframework.http.server.reactive.ContextPathCompositeHandler;
 import org.springframework.http.server.reactive.HttpHandler;
 import org.springframework.util.Assert;
+import org.springframework.util.StopWatch;
 
 /**
  * @author Rossen Stoyanchev
  */
 public abstract class AbstractHttpServer implements HttpServer {
 
+	protected Log logger = LogFactory.getLog(getClass().getName());
+
 	private String host = "0.0.0.0";
 
 	private int port = 0;
@@ -116,9 +122,15 @@ public abstract class AbstractHttpServer implements HttpServer {
 	public final void start() {
 		synchronized (this.lifecycleMonitor) {
 			if (!isRunning()) {
+				String serverName = getClass().getSimpleName();
+				logger.debug("Starting " + serverName + "...");
 				this.running = true;
 				try {
+					StopWatch stopWatch = new StopWatch();
+					stopWatch.start();
 					startInternal();
+					long millis = stopWatch.getTotalTimeMillis();
+					logger.debug("Server started on port " + getPort() + "(" + millis + " millis).");
 				}
 				catch (Throwable ex) {
 					throw new IllegalStateException(ex);
@@ -134,9 +146,14 @@ public abstract class AbstractHttpServer implements HttpServer {
 	public final void stop() {
 		synchronized (this.lifecycleMonitor) {
 			if (isRunning()) {
+				String serverName = getClass().getSimpleName();
+				logger.debug("Stopping " + serverName + "...");
 				this.running = false;
 				try {
+					StopWatch stopWatch = new StopWatch();
+					stopWatch.start();
 					stopInternal();
+					logger.debug("Server stopped (" + stopWatch.getTotalTimeMillis() + " millis).");
 				}
 				catch (Throwable ex) {
 					throw new IllegalStateException(ex);