Skip to content

Commit

Permalink
Merge branch '2.5.x' into 2.6.x
Browse files Browse the repository at this point in the history
Closes gh-29454
  • Loading branch information
wilkinsona committed Jan 18, 2022
2 parents d28942e + 1c6471e commit 4ee260a
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 6 deletions.
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2021 the original author or authors.
* Copyright 2012-2022 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -72,7 +72,9 @@ public class TomcatReactiveWebServerFactory extends AbstractReactiveWebServerFac

private final List<Valve> engineValves = new ArrayList<>();

private List<LifecycleListener> contextLifecycleListeners = getDefaultLifecycleListeners();
private List<LifecycleListener> contextLifecycleListeners = new ArrayList<>();

private List<LifecycleListener> serverLifecycleListeners = getDefaultServerLifecycleListeners();

private Set<TomcatContextCustomizer> tomcatContextCustomizers = new LinkedHashSet<>();

Expand Down Expand Up @@ -105,7 +107,7 @@ public TomcatReactiveWebServerFactory(int port) {
super(port);
}

private static List<LifecycleListener> getDefaultLifecycleListeners() {
private static List<LifecycleListener> getDefaultServerLifecycleListeners() {
AprLifecycleListener aprLifecycleListener = new AprLifecycleListener();
return AprLifecycleListener.isAprAvailable() ? new ArrayList<>(Arrays.asList(aprLifecycleListener))
: new ArrayList<>();
Expand All @@ -119,6 +121,9 @@ public WebServer getWebServer(HttpHandler httpHandler) {
Tomcat tomcat = new Tomcat();
File baseDir = (this.baseDirectory != null) ? this.baseDirectory : createTempDir("tomcat");
tomcat.setBaseDir(baseDir.getAbsolutePath());
for (LifecycleListener listener : this.serverLifecycleListeners) {
tomcat.getServer().addLifecycleListener(listener);
}
Connector connector = new Connector(this.protocol);
connector.setThrowOnFailure(true);
tomcat.getService().addConnector(connector);
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2021 the original author or authors.
* Copyright 2012-2022 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -124,7 +124,9 @@ public class TomcatServletWebServerFactory extends AbstractServletWebServerFacto

private List<Valve> contextValves = new ArrayList<>();

private List<LifecycleListener> contextLifecycleListeners = getDefaultLifecycleListeners();
private List<LifecycleListener> contextLifecycleListeners = new ArrayList<>();

private List<LifecycleListener> serverLifecycleListeners = getDefaultServerLifecycleListeners();

private Set<TomcatContextCustomizer> tomcatContextCustomizers = new LinkedHashSet<>();

Expand Down Expand Up @@ -173,7 +175,7 @@ public TomcatServletWebServerFactory(String contextPath, int port) {
super(contextPath, port);
}

private static List<LifecycleListener> getDefaultLifecycleListeners() {
private static List<LifecycleListener> getDefaultServerLifecycleListeners() {
ArrayList<LifecycleListener> lifecycleListeners = new ArrayList<>();
if (!NativeDetector.inNativeImage()) {
AprLifecycleListener aprLifecycleListener = new AprLifecycleListener();
Expand All @@ -192,6 +194,9 @@ public WebServer getWebServer(ServletContextInitializer... initializers) {
Tomcat tomcat = new Tomcat();
File baseDir = (this.baseDirectory != null) ? this.baseDirectory : createTempDir("tomcat");
tomcat.setBaseDir(baseDir.getAbsolutePath());
for (LifecycleListener listener : this.serverLifecycleListeners) {
tomcat.getServer().addLifecycleListener(listener);
}
Connector connector = new Connector(this.protocol);
connector.setThrowOnFailure(true);
tomcat.getService().addConnector(connector);
Expand Down

0 comments on commit 4ee260a

Please sign in to comment.