Adapt to deprecation of rejectIllegalHeader

Closes gh-35442
This commit is contained in:
Andy Wilkinson
2023-05-16 19:46:43 +01:00
parent c631f04391
commit f5ff8007be
3 changed files with 5 additions and 0 deletions

View File

@@ -459,6 +459,7 @@ public class ServerProperties {
/** /**
* Whether to reject requests with illegal header names or values. * Whether to reject requests with illegal header names or values.
*/ */
@Deprecated
private boolean rejectIllegalHeader = true; private boolean rejectIllegalHeader = true;
/** /**
@@ -612,6 +613,7 @@ public class ServerProperties {
this.connectionTimeout = connectionTimeout; this.connectionTimeout = connectionTimeout;
} }
@DeprecatedConfigurationProperty(reason = "The setting has been deprecated in Tomcat")
public boolean isRejectIllegalHeader() { public boolean isRejectIllegalHeader() {
return this.rejectIllegalHeader; return this.rejectIllegalHeader;
} }

View File

@@ -220,6 +220,7 @@ public class TomcatWebServerFactoryCustomizer
factory.addConnectorCustomizers((connector) -> connector.setProperty("relaxedQueryChars", relaxedChars)); factory.addConnectorCustomizers((connector) -> connector.setProperty("relaxedQueryChars", relaxedChars));
} }
@SuppressWarnings("deprecation")
private void customizeRejectIllegalHeader(ConfigurableTomcatWebServerFactory factory, boolean rejectIllegalHeader) { private void customizeRejectIllegalHeader(ConfigurableTomcatWebServerFactory factory, boolean rejectIllegalHeader) {
factory.addConnectorCustomizers((connector) -> { factory.addConnectorCustomizers((connector) -> {
ProtocolHandler handler = connector.getProtocolHandler(); ProtocolHandler handler = connector.getProtocolHandler();

View File

@@ -365,6 +365,8 @@ class TomcatWebServerFactoryCustomizerTests {
} }
@Test @Test
@Deprecated
@SuppressWarnings("deprecation")
void testCustomizeRejectIllegalHeader() { void testCustomizeRejectIllegalHeader() {
bind("server.tomcat.reject-illegal-header=false"); bind("server.tomcat.reject-illegal-header=false");
customizeAndRunServer((server) -> assertThat( customizeAndRunServer((server) -> assertThat(