Polish SecurityProperties

See gh-33251
This commit is contained in:
Vedran Pavic 2022-11-18 12:00:00 +01:00 committed by Andy Wilkinson
parent 3c1c91bc5d
commit c1691f525a
2 changed files with 4 additions and 6 deletions

View File

@ -17,8 +17,7 @@
package org.springframework.boot.autoconfigure.security; package org.springframework.boot.autoconfigure.security;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Arrays; import java.util.EnumSet;
import java.util.HashSet;
import java.util.List; import java.util.List;
import java.util.Set; import java.util.Set;
import java.util.UUID; import java.util.UUID;
@ -83,8 +82,7 @@ public class SecurityProperties {
/** /**
* Security filter chain dispatcher types. * Security filter chain dispatcher types.
*/ */
private Set<DispatcherType> dispatcherTypes = new HashSet<>(Arrays.asList(DispatcherType.ASYNC, private Set<DispatcherType> dispatcherTypes = EnumSet.allOf(DispatcherType.class);
DispatcherType.ERROR, DispatcherType.REQUEST, DispatcherType.FORWARD, DispatcherType.INCLUDE));
public int getOrder() { public int getOrder() {
return this.order; return this.order;

View File

@ -17,6 +17,7 @@
package org.springframework.boot.autoconfigure.security.servlet; package org.springframework.boot.autoconfigure.security.servlet;
import java.security.interfaces.RSAPublicKey; import java.security.interfaces.RSAPublicKey;
import java.util.EnumSet;
import jakarta.servlet.DispatcherType; import jakarta.servlet.DispatcherType;
import org.assertj.core.api.InstanceOfAssertFactories; import org.assertj.core.api.InstanceOfAssertFactories;
@ -161,8 +162,7 @@ class SecurityAutoConfigurationTests {
DelegatingFilterProxyRegistrationBean.class); DelegatingFilterProxyRegistrationBean.class);
assertThat(bean) assertThat(bean)
.extracting("dispatcherTypes", InstanceOfAssertFactories.iterable(DispatcherType.class)) .extracting("dispatcherTypes", InstanceOfAssertFactories.iterable(DispatcherType.class))
.containsOnly(DispatcherType.ASYNC, DispatcherType.ERROR, DispatcherType.REQUEST, .containsOnly(EnumSet.allOf(DispatcherType.class).toArray(new DispatcherType[0]));
DispatcherType.INCLUDE, DispatcherType.FORWARD);
}); });
} }