add request id filter to all request logs with missing xid header

guardian · Feb 5, 2025 · a464a90 · a464a90
1 parent 4643d19
commit a464a90
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/common/app/http/Filters.scala b/common/app/http/Filters.scala
@@ -123,6 +123,7 @@ object Filters {
       executionContext: ExecutionContext,
   ): List[EssentialFilter] =
     List(
+      new RequestIdFilter(),
       new RequestLoggingFilter,
       new PanicSheddingFilter,
       new JsonVaryHeadersFilter,

diff --git a/preview/app/AppLoader.scala b/preview/app/AppLoader.scala
@@ -18,14 +18,7 @@ import cricket.controllers.CricketControllers
 import dev.DevAssetsController
 import feed.OnwardJourneyLifecycle
 import football.controllers.FootballControllers
-import http.{
-  Filters,
-  GuardianAuthWithExemptions,
-  PreviewContentSecurityPolicyFilter,
-  PreviewNoCacheFilter,
-  RequestIdFilter,
-  routes,
-}
+import http.{Filters, GuardianAuthWithExemptions, PreviewContentSecurityPolicyFilter, PreviewNoCacheFilter, routes}
 import model.ApplicationIdentity
 import play.api.ApplicationLoader.Context
 import play.api.http.HttpErrorHandler
@@ -143,7 +136,7 @@ trait AppComponents
     standaloneLifecycleComponents :+ wire[CachedHealthCheckLifeCycle]
 
   override lazy val httpFilters: Seq[EssentialFilter] =
-    new PreviewNoCacheFilter :: new PreviewContentSecurityPolicyFilter :: new RequestIdFilter :: Filters.common(
+    new PreviewNoCacheFilter :: new PreviewContentSecurityPolicyFilter :: Filters.common(
       frontend.preview.BuildInfo,
     )