Commit 3ff20b24 authored by Phillip Webb's avatar Phillip Webb

Restore import-control checkstyle

Restore checkstyle import control and fix a few violations in the
process.

Closes gh-14660
parent 63b64b4e
...@@ -53,6 +53,7 @@ ...@@ -53,6 +53,7 @@
<configLocation>src/checkstyle/checkstyle.xml</configLocation> <configLocation>src/checkstyle/checkstyle.xml</configLocation>
<suppressionsLocation>src/checkstyle/checkstyle-suppressions.xml</suppressionsLocation> <suppressionsLocation>src/checkstyle/checkstyle-suppressions.xml</suppressionsLocation>
<includeTestSourceDirectory>true</includeTestSourceDirectory> <includeTestSourceDirectory>true</includeTestSourceDirectory>
<propertyExpansion>main.basedir=${main.basedir}</propertyExpansion>
</configuration> </configuration>
<goals> <goals>
<goal>check</goal> <goal>check</goal>
......
...@@ -12,7 +12,7 @@ ...@@ -12,7 +12,7 @@
<name>Spring Boot Build</name> <name>Spring Boot Build</name>
<description>Spring Boot Build</description> <description>Spring Boot Build</description>
<properties> <properties>
<main.basedir>${basedir}/../..</main.basedir> <main.basedir>${basedir}/..</main.basedir>
</properties> </properties>
<modules> <modules>
<module>spring-boot-dependencies</module> <module>spring-boot-dependencies</module>
......
...@@ -31,6 +31,7 @@ ...@@ -31,6 +31,7 @@
</developer> </developer>
</developers> </developers>
<properties> <properties>
<main.basedir>${basedir}/../..</main.basedir>
<!-- Dependency versions --> <!-- Dependency versions -->
<activemq.version>5.15.6</activemq.version> <activemq.version>5.15.6</activemq.version>
<antlr2.version>2.7.7</antlr2.version> <antlr2.version>2.7.7</antlr2.version>
......
...@@ -14,6 +14,7 @@ ...@@ -14,6 +14,7 @@
<description>Parent pom providing dependency and plugin management for applications <description>Parent pom providing dependency and plugin management for applications
built with Maven</description> built with Maven</description>
<properties> <properties>
<main.basedir>${basedir}/../../..</main.basedir>
<java.version>1.8</java.version> <java.version>1.8</java.version>
<resource.delimiter>@</resource.delimiter> <!-- delimiter that doesn't clash with Spring ${} placeholders --> <resource.delimiter>@</resource.delimiter> <!-- delimiter that doesn't clash with Spring ${} placeholders -->
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
package org.springframework.boot.web.reactive.filter; package org.springframework.boot.web.reactive.filter;
import org.springframework.boot.web.servlet.FilterRegistrationBean;
import org.springframework.core.Ordered; import org.springframework.core.Ordered;
import org.springframework.web.filter.reactive.HiddenHttpMethodFilter; import org.springframework.web.filter.reactive.HiddenHttpMethodFilter;
...@@ -32,8 +31,7 @@ public class OrderedHiddenHttpMethodFilter extends HiddenHttpMethodFilter ...@@ -32,8 +31,7 @@ public class OrderedHiddenHttpMethodFilter extends HiddenHttpMethodFilter
/** /**
* The default order is high to ensure the filter is applied before Spring Security. * The default order is high to ensure the filter is applied before Spring Security.
*/ */
public static final int DEFAULT_ORDER = FilterRegistrationBean.REQUEST_WRAPPER_FILTER_MAX_ORDER public static final int DEFAULT_ORDER = -10000;
- 10000;
private int order = DEFAULT_ORDER; private int order = DEFAULT_ORDER;
......
...@@ -11,5 +11,11 @@ ...@@ -11,5 +11,11 @@
<property name="illegalClasses" <property name="illegalClasses"
value="^reactor\.core\.support\.Assert" /> value="^reactor\.core\.support\.Assert" />
</module> </module>
<module
name="com.puppycrawl.tools.checkstyle.checks.imports.ImportControlCheck">
<property name="file"
value="${main.basedir}/src/checkstyle/import-control.xml" />
<property name="path" value="^.*[\\/]src[\\/]main[\\/].*$" />
</module>
</module> </module>
</module> </module>
...@@ -5,12 +5,12 @@ ...@@ -5,12 +5,12 @@
<subpackage name="autoconfigure"> <subpackage name="autoconfigure">
<subpackage name="web"> <subpackage name="web">
<allow pkg="org.springframework.boot.web.server" />
<allow pkg="org.springframework.boot.web.servlet.server" />
<disallow pkg="org.springframework.boot.web" /> <disallow pkg="org.springframework.boot.web" />
<disallow pkg="org.springframework.web.servlet" /> <disallow pkg="org.springframework.web.servlet" />
<disallow pkg="org.springframework.web.reactive" /> <disallow pkg="org.springframework.web.reactive" />
<disallow pkg="javax.servlet" /> <disallow pkg="javax.servlet" />
<allow pkg="org.springframework.boot.web.server" />
<allow pkg="org.springframework.boot.web.servlet.server" />
<subpackage name="client"> <subpackage name="client">
<allow pkg="org.springframework.boot.web.client" /> <allow pkg="org.springframework.boot.web.client" />
</subpackage> </subpackage>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment