Skip to content

Commit 819529f

Browse files
author
Steve Riesenberg
committed
Remove CsrfSpec.tokenFromMultipartDataEnabled
Also removed ServerCsrfDsl.tokenFromMultipartDataEnabled Closes gh-12020
1 parent db7732d commit 819529f

File tree

3 files changed

+3
-24
lines changed

3 files changed

+3
-24
lines changed

config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,6 @@
149149
import org.springframework.security.web.server.csrf.CsrfServerLogoutHandler;
150150
import org.springframework.security.web.server.csrf.CsrfWebFilter;
151151
import org.springframework.security.web.server.csrf.ServerCsrfTokenRepository;
152-
import org.springframework.security.web.server.csrf.ServerCsrfTokenRequestAttributeHandler;
153152
import org.springframework.security.web.server.csrf.ServerCsrfTokenRequestHandler;
154153
import org.springframework.security.web.server.csrf.WebSessionServerCsrfTokenRepository;
155154
import org.springframework.security.web.server.header.CacheControlServerHttpHeadersWriter;
@@ -1865,22 +1864,6 @@ public CsrfSpec requireCsrfProtectionMatcher(ServerWebExchangeMatcher requireCsr
18651864
return this;
18661865
}
18671866

1868-
/**
1869-
* Specifies if {@link CsrfWebFilter} should try to resolve the actual CSRF token
1870-
* from the body of multipart data requests.
1871-
* @param enabled true if should read from multipart form body, else false.
1872-
* Default is false
1873-
* @return the {@link CsrfSpec} for additional configuration
1874-
* @deprecated Use
1875-
* {@link ServerCsrfTokenRequestAttributeHandler#setTokenFromMultipartDataEnabled(boolean)}
1876-
* instead
1877-
*/
1878-
@Deprecated
1879-
public CsrfSpec tokenFromMultipartDataEnabled(boolean enabled) {
1880-
this.filter.setTokenFromMultipartDataEnabled(enabled);
1881-
return this;
1882-
}
1883-
18841867
/**
18851868
* Specifies a {@link ServerCsrfTokenRequestHandler} that is used to make the
18861869
* {@code CsrfToken} available as an exchange attribute.

config/src/main/kotlin/org/springframework/security/config/web/server/ServerCsrfDsl.kt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
package org.springframework.security.config.web.server
1818

1919
import org.springframework.security.web.server.authorization.ServerAccessDeniedHandler
20-
import org.springframework.security.web.server.csrf.CsrfWebFilter
2120
import org.springframework.security.web.server.csrf.ServerCsrfTokenRepository
2221
import org.springframework.security.web.server.csrf.ServerCsrfTokenRequestHandler
2322
import org.springframework.security.web.server.util.matcher.ServerWebExchangeMatcher
@@ -32,8 +31,6 @@ import org.springframework.security.web.server.util.matcher.ServerWebExchangeMat
3231
* @property csrfTokenRepository the [ServerCsrfTokenRepository] used to persist the CSRF token.
3332
* @property requireCsrfProtectionMatcher the [ServerWebExchangeMatcher] used to determine when CSRF protection
3433
* is enabled.
35-
* @property tokenFromMultipartDataEnabled if true, the [CsrfWebFilter] should try to resolve the actual CSRF
36-
* token from the body of multipart data requests.
3734
* @property csrfTokenRequestHandler the [ServerCsrfTokenRequestHandler] that is used to make the CSRF token
3835
* available as an exchange attribute
3936
*/
@@ -42,8 +39,6 @@ class ServerCsrfDsl {
4239
var accessDeniedHandler: ServerAccessDeniedHandler? = null
4340
var csrfTokenRepository: ServerCsrfTokenRepository? = null
4441
var requireCsrfProtectionMatcher: ServerWebExchangeMatcher? = null
45-
@Deprecated("Use 'csrfTokenRequestHandler' instead")
46-
var tokenFromMultipartDataEnabled: Boolean? = null
4742
var csrfTokenRequestHandler: ServerCsrfTokenRequestHandler? = null
4843

4944
private var disabled = false
@@ -60,7 +55,6 @@ class ServerCsrfDsl {
6055
accessDeniedHandler?.also { csrf.accessDeniedHandler(accessDeniedHandler) }
6156
csrfTokenRepository?.also { csrf.csrfTokenRepository(csrfTokenRepository) }
6257
requireCsrfProtectionMatcher?.also { csrf.requireCsrfProtectionMatcher(requireCsrfProtectionMatcher) }
63-
tokenFromMultipartDataEnabled?.also { csrf.tokenFromMultipartDataEnabled(tokenFromMultipartDataEnabled!!) }
6458
csrfTokenRequestHandler?.also { csrf.csrfTokenRequestHandler(csrfTokenRequestHandler) }
6559
if (disabled) {
6660
csrf.disable()

config/src/test/kotlin/org/springframework/security/config/web/server/ServerCsrfDslTests.kt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,9 @@ class ServerCsrfDslTests {
311311
return http {
312312
csrf {
313313
csrfTokenRepository = TOKEN_REPOSITORY
314-
tokenFromMultipartDataEnabled = true
314+
csrfTokenRequestHandler = XorServerCsrfTokenRequestAttributeHandler().apply {
315+
setTokenFromMultipartDataEnabled(true)
316+
}
315317
}
316318
}
317319
}

0 commit comments

Comments
 (0)