Allow configuring custom ServerHttpHeadersWriter for Kotlin DSL

Closes gh-16009
This commit is contained in:
Evgeniy Cheban 2024-11-20 06:09:14 +02:00 committed by Josh Cummings
parent 12a6431496
commit f45cc22e11
2 changed files with 62 additions and 2 deletions

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2024 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -17,6 +17,7 @@
package org.springframework.security.config.web.server package org.springframework.security.config.web.server
import org.springframework.security.web.server.header.CacheControlServerHttpHeadersWriter import org.springframework.security.web.server.header.CacheControlServerHttpHeadersWriter
import org.springframework.security.web.server.header.ServerHttpHeadersWriter
import org.springframework.security.web.server.header.ContentTypeOptionsServerHttpHeadersWriter import org.springframework.security.web.server.header.ContentTypeOptionsServerHttpHeadersWriter
import org.springframework.security.web.server.header.ReferrerPolicyServerHttpHeadersWriter import org.springframework.security.web.server.header.ReferrerPolicyServerHttpHeadersWriter
import org.springframework.security.web.server.header.StrictTransportSecurityServerHttpHeadersWriter import org.springframework.security.web.server.header.StrictTransportSecurityServerHttpHeadersWriter
@ -43,6 +44,7 @@ class ServerHeadersDsl {
private var crossOriginOpenerPolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginOpenerPolicySpec) -> Unit)? = null private var crossOriginOpenerPolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginOpenerPolicySpec) -> Unit)? = null
private var crossOriginEmbedderPolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginEmbedderPolicySpec) -> Unit)? = null private var crossOriginEmbedderPolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginEmbedderPolicySpec) -> Unit)? = null
private var crossOriginResourcePolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginResourcePolicySpec) -> Unit)? = null private var crossOriginResourcePolicy: ((ServerHttpSecurity.HeaderSpec.CrossOriginResourcePolicySpec) -> Unit)? = null
private var writers = mutableListOf<ServerHttpHeadersWriter>()
private var disabled = false private var disabled = false
@ -198,6 +200,16 @@ class ServerHeadersDsl {
this.crossOriginResourcePolicy = ServerCrossOriginResourcePolicyDsl().apply(crossOriginResourcePolicyConfig).get() this.crossOriginResourcePolicy = ServerCrossOriginResourcePolicyDsl().apply(crossOriginResourcePolicyConfig).get()
} }
/**
* Configures custom headers writer
*
* @since 6.5
* @param writer the [ServerHttpHeadersWriter] to provide custom headers writer
*/
fun writer(writer: ServerHttpHeadersWriter) {
this.writers.add(writer)
}
/** /**
* Disables HTTP response headers. * Disables HTTP response headers.
*/ */
@ -244,6 +256,9 @@ class ServerHeadersDsl {
crossOriginResourcePolicy?.also { crossOriginResourcePolicy?.also {
headers.crossOriginResourcePolicy(crossOriginResourcePolicy) headers.crossOriginResourcePolicy(crossOriginResourcePolicy)
} }
writers.also {
writers.forEach { writer -> headers.writer(writer) }
}
if (disabled) { if (disabled) {
headers.disable() headers.disable()
} }

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2024 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -37,6 +37,7 @@ import org.springframework.security.web.server.header.XFrameOptionsServerHttpHea
import org.springframework.security.web.server.header.XXssProtectionServerHttpHeadersWriter import org.springframework.security.web.server.header.XXssProtectionServerHttpHeadersWriter
import org.springframework.test.web.reactive.server.WebTestClient import org.springframework.test.web.reactive.server.WebTestClient
import org.springframework.web.reactive.config.EnableWebFlux import org.springframework.web.reactive.config.EnableWebFlux
import reactor.core.publisher.Mono
/** /**
* Tests for [ServerHeadersDsl] * Tests for [ServerHeadersDsl]
@ -198,4 +199,48 @@ class ServerHeadersDslTests {
} }
} }
} }
@Test
fun `request when custom server http headers writer configured then custom http headers added`() {
this.spring.register(ServerHttpHeadersWriterCustomConfig::class.java).autowire()
this.client.get()
.uri("/")
.exchange()
.expectHeader().valueEquals("CUSTOM-HEADER-1", "CUSTOM-VALUE-1")
.expectHeader().valueEquals("CUSTOM-HEADER-2", "CUSTOM-VALUE-2")
}
@Configuration
@EnableWebFluxSecurity
@EnableWebFlux
open class ServerHttpHeadersWriterCustomConfig {
@Bean
open fun springWebFilterChain(http: ServerHttpSecurity): SecurityWebFilterChain {
return http {
headers {
writer { exchange ->
Mono.just(exchange)
.doOnNext {
it.response.headers.add(
"CUSTOM-HEADER-1",
"CUSTOM-VALUE-1"
)
}
.then()
}
writer { exchange ->
Mono.just(exchange)
.doOnNext {
it.response.headers.add(
"CUSTOM-HEADER-2",
"CUSTOM-VALUE-2"
)
}
.then()
}
}
}
}
}
} }