SEC-2749: CsrfConfigurer.requireCsrfProtectionMatcher correct null check
This commit is contained in:
parent
4d738d8576
commit
1cca72e6d8
|
@ -97,7 +97,7 @@ public final class CsrfConfigurer<H extends HttpSecurityBuilder<H>> extends Abst
|
||||||
* @return the {@link CsrfConfigurer} for further customizations
|
* @return the {@link CsrfConfigurer} for further customizations
|
||||||
*/
|
*/
|
||||||
public CsrfConfigurer<H> requireCsrfProtectionMatcher(RequestMatcher requireCsrfProtectionMatcher) {
|
public CsrfConfigurer<H> requireCsrfProtectionMatcher(RequestMatcher requireCsrfProtectionMatcher) {
|
||||||
Assert.notNull(csrfTokenRepository, "requireCsrfProtectionMatcher cannot be null");
|
Assert.notNull(requireCsrfProtectionMatcher, "requireCsrfProtectionMatcher cannot be null");
|
||||||
this.requireCsrfProtectionMatcher = requireCsrfProtectionMatcher;
|
this.requireCsrfProtectionMatcher = requireCsrfProtectionMatcher;
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
|
|
|
@ -464,6 +464,13 @@ class CsrfConfigurerTests extends BaseSpringSpec {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
def 'SEC-2749: requireCsrfProtectionMatcher null'() {
|
||||||
|
when:
|
||||||
|
new CsrfConfigurer<>().requireCsrfProtectionMatcher(null)
|
||||||
|
then:
|
||||||
|
thrown(IllegalArgumentException)
|
||||||
|
}
|
||||||
|
|
||||||
def clearCsrfToken() {
|
def clearCsrfToken() {
|
||||||
request.removeAllParameters()
|
request.removeAllParameters()
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue