From c2ca16251873670ea29148e86008d6c17135f5c2 Mon Sep 17 00:00:00 2001 From: david-leifker <114954101+david-leifker@users.noreply.github.com> Date: Tue, 18 Jun 2024 12:42:06 -0500 Subject: [PATCH] config(header): increase header size to 32k (#10743) --- datahub-frontend/conf/application.conf | 6 +++++- docker/datahub-gms/jetty.xml | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/datahub-frontend/conf/application.conf b/datahub-frontend/conf/application.conf index 045175ba69..dc243ecada 100644 --- a/datahub-frontend/conf/application.conf +++ b/datahub-frontend/conf/application.conf @@ -38,8 +38,12 @@ jwt { play.server.provider = server.CustomAkkaHttpServerProvider play.http.server.akka.max-header-count = 64 play.http.server.akka.max-header-count = ${?DATAHUB_AKKA_MAX_HEADER_COUNT} -play.server.akka.max-header-size = 8k +# max-header-size is reportedly no longer used +play.server.akka.max-header-size = 32k play.server.akka.max-header-size = ${?DATAHUB_AKKA_MAX_HEADER_VALUE_LENGTH} +# max header value length seems to impact the actual limit +play.server.akka.max-header-value-length = 32k +play.server.akka.max-header-value-length = ${?DATAHUB_AKKA_MAX_HEADER_VALUE_LENGTH} # Update AUTH_COOKIE_SAME_SITE and AUTH_COOKIE_SECURE in order to change how authentication cookies # are configured. If you wish cookies to be sent in first and third party contexts, set diff --git a/docker/datahub-gms/jetty.xml b/docker/datahub-gms/jetty.xml index 589442b115..3f04635d94 100644 --- a/docker/datahub-gms/jetty.xml +++ b/docker/datahub-gms/jetty.xml @@ -31,7 +31,7 @@ - +