Skip to content

Commit 5cc0061

Browse files
committed
improved session cookie config
1 parent d7b4e74 commit 5cc0061

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

dsf-common/dsf-common-jetty/src/main/java/dev/dsf/common/config/AbstractJettyConfig.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import org.eclipse.jetty.client.transport.HttpClientTransportOverHTTP;
4343
import org.eclipse.jetty.ee11.servlet.SessionHandler;
4444
import org.eclipse.jetty.ee11.webapp.WebAppContext;
45+
import org.eclipse.jetty.http.HttpCookie.SameSite;
4546
import org.eclipse.jetty.http.HttpField;
4647
import org.eclipse.jetty.http.HttpHeader;
4748
import org.eclipse.jetty.io.ClientConnector;
@@ -91,6 +92,7 @@
9192
import dev.dsf.common.oidc.JwtVerifier;
9293
import dev.dsf.common.oidc.JwtVerifierImpl;
9394
import jakarta.servlet.ServletContainerInitializer;
95+
import jakarta.servlet.SessionCookieConfig;
9496

9597
@Configuration
9698
@PropertySource(value = "file:conf/jetty.properties", encoding = "UTF-8", ignoreResourceNotFound = true)
@@ -315,6 +317,12 @@ private KeyStore clientCertificateTrustStore()
315317
private void configureSecurityHandler(WebAppContext webAppContext, Supplier<Integer> statusPortSupplier)
316318
{
317319
SessionHandler sessionHandler = webAppContext.getSessionHandler();
320+
sessionHandler.setSameSite(SameSite.LAX);
321+
322+
SessionCookieConfig sessionCookieConfig = sessionHandler.getSessionCookieConfig();
323+
sessionCookieConfig.setSecure(true);
324+
sessionCookieConfig.setHttpOnly(true);
325+
318326
DsfLoginService dsfLoginService = new DsfLoginService(webAppContext);
319327

320328
OpenIdConfiguration openIdConfiguration = null;

0 commit comments

Comments
 (0)