Skip to content

Commit f5a5121

Browse files
Revert tomcat v10.1.42 update (#1102)
1 parent d058b30 commit f5a5121

5 files changed

Lines changed: 1 addition & 44 deletions

File tree

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ apacheDirectoryVersion=2.1.7
9999
apacheMinaVersion=2.2.4
100100

101101
# Usually matches the version specified as a Spring Boot dependency (see springBootVersion below)
102-
apacheTomcatVersion=10.1.42
102+
apacheTomcatVersion=10.1.41
103103

104104
# (mothership) -> json-path -> json-smart -> accessor-smart
105105
# (core) -> graalvm

server/configs/application.properties

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,6 @@ context.encryptionKey=@@encryptionKey@@
6565
#context.bypass2FA=true
6666
#context.workDirLocation=/path/to/desired/workDir
6767

68-
## Tomcat v10.1.42 lowered the default for part count from 1000 to 10. Our default is now 500, but can be overridden here.
69-
## Header size default changed from 10Kb to 512, which is also our default.
70-
#context.maxConnectorPartCount=500
71-
#context.maxConnectorPartHeaderSize=512
72-
7368
## SMTP configuration
7469
mail.smtpHost=@@smtpHost@@
7570
mail.smtpPort=@@smtpPort@@

server/configs/webapps/embedded/config/application.properties

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,11 +103,6 @@ mail.smtpUser=Anonymous
103103
#context.bypass2FA=true
104104
#context.workDirLocation=@@/path/to/desired/workDir@@
105105

106-
## Tomcat v10.1.42 lowered the default for part count from 1000 to 10. Our default is now 500, but can be overridden here.
107-
## Header size default changed from 10Kb to 512, which is also our default.
108-
#context.maxConnectorPartCount=500
109-
#context.maxConnectorPartHeaderSize=512
110-
111106
## Other webapps to be deployed, most commonly to deliver a set of static files. The context path to deploy into is the
112107
## property name after the "context.additionalWebapps." prefix, and the value is the location of the webapp on disk
113108
#context.additionalWebapps.firstContextPath=@@/my/webapp/path@@

server/embedded/src/org/labkey/embedded/LabKeyServer.java

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -148,14 +148,6 @@ public WebServerFactoryCustomizer<TomcatServletWebServerFactory> customizer()
148148
return customizer -> customizer.setDisableMBeanRegistry(false);
149149
}
150150

151-
@Bean
152-
TomcatConnectorCustomizer connectorCustomizer() {
153-
return (connector) -> {
154-
connector.setMaxPartCount(contextSource().getMaxConnectorPartCount());
155-
connector.setMaxPartHeaderSize(contextSource().getMaxConnectorPartHeaderSize());
156-
};
157-
}
158-
159151
@Bean
160152
public TomcatServletWebServerFactory servletContainerFactory()
161153
{
@@ -168,7 +160,6 @@ public TomcatServletWebServerFactory servletContainerFactory()
168160
Connector httpConnector = new Connector();
169161
httpConnector.setScheme("http");
170162
httpConnector.setPort(contextProperties.getHttpPort());
171-
result.getTomcatConnectorCustomizers().forEach(customizer -> customizer.customize(httpConnector));
172163
result.addAdditionalTomcatConnectors(httpConnector);
173164
}
174165

@@ -466,9 +457,6 @@ public static class ContextProperties
466457
private Map<String, Map<String, Map<String, String>>> resources;
467458
private Map<String, String> additionalWebapps;
468459

469-
private Integer maxConnectorPartCount = 500;
470-
private Integer maxConnectorPartHeaderSize = 512;
471-
472460
public List<String> getDataSourceName()
473461
{
474462
return dataSourceName;
@@ -731,26 +719,6 @@ public void setAdditionalWebapps(Map<String, String> additionalWebapps)
731719
{
732720
this.additionalWebapps = additionalWebapps;
733721
}
734-
735-
public Integer getMaxConnectorPartCount()
736-
{
737-
return maxConnectorPartCount;
738-
}
739-
740-
public void setMaxConnectorPartCount(Integer maxConnectorPartCount)
741-
{
742-
this.maxConnectorPartCount = maxConnectorPartCount;
743-
}
744-
745-
public Integer getMaxConnectorPartHeaderSize()
746-
{
747-
return maxConnectorPartHeaderSize;
748-
}
749-
750-
public void setMaxConnectorPartHeaderSize(Integer maxConnectorPartHeaderSize)
751-
{
752-
this.maxConnectorPartHeaderSize = maxConnectorPartHeaderSize;
753-
}
754722
}
755723

756724
@Configuration

server/embedded/src/org/labkey/embedded/LabKeyTomcatServletWebServerFactory.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ public LabKeyTomcatServletWebServerFactory(LabKeyServer server)
3838

3939
addConnectorCustomizers(connector -> {
4040
LabKeyServer.TomcatProperties props = _server.tomcatProperties();
41-
_server.connectorCustomizer().customize(connector);
4241

4342
if (props.getUseBodyEncodingForURI() != null)
4443
{

0 commit comments

Comments
 (0)