Include header in the file/function name for consistency

Author: karinathomasbbc

src/server/index.jsx

@@ -34,7 +34,7 @@ import getAgent from './utilities/getAgent';
 import { getMvtExperiments, getMvtVaryHeaders } from './utilities/mvtHeader';
 import getAssetOrigins from './utilities/getAssetOrigins';
 import extractHeaders from './utilities/extractHeaders';
-import addPlatformToRequestChain from './utilities/addPlatformToRequestChain';
+import addPlatformToRequestChainHeader from './utilities/addPlatformToRequestChainHeader';
 
 const morgan = require('morgan');
 
@@ -161,8 +161,11 @@ const injectDefaultCacheHeader = (req, res, next) => {
   next();
 };
 
-const injectPlatformToRequestChain = (req, res, next) => {
-  res.set('req-svc-chain', addPlatformToRequestChain({ headers: req.headers }));
+const injectPlatformToRequestChainHeader = (req, res, next) => {
+  res.set(
+    'req-svc-chain',
+    addPlatformToRequestChainHeader({ headers: req.headers }),
+  );
   next();
 };
 
@@ -198,7 +201,7 @@ server.get(
     injectDefaultCacheHeader,
     injectReferrerPolicyHeader,
     injectResourceHintsHeader,
-    injectPlatformToRequestChain,
+    injectPlatformToRequestChainHeader,
   ],
   async ({ url, query, headers, path: urlPath }, res) => {
     let derivedPageType = 'Unknown';

src/server/utilities/addPlatformToRequestChain/index.test.ts src/server/utilities/addPlatformToRequestChainHeader/index.test.ts

@@ -1,9 +1,9 @@
-import addPlatformToRequestChain from '.';
+import addPlatformToRequestChainHeader from '.';
 
 describe('addPlatformToRequestChain', () => {
   it('appends SIMORGH to the req-svc-chain header if it exists', () => {
     const headers = { 'req-svc-chain': 'SERVICE_1,SERVICE_2' };
-    const result = addPlatformToRequestChain({
+    const result = addPlatformToRequestChainHeader({
       headers,
     });
 
@@ -12,7 +12,7 @@ describe('addPlatformToRequestChain', () => {
 
   it('does not append SIMORGH to the req-svc-chain header if it is the final service', () => {
     const headers = { 'req-svc-chain': 'SERVICE_1,SERVICE_2,SIMORGH' };
-    const result = addPlatformToRequestChain({
+    const result = addPlatformToRequestChainHeader({
       headers,
     });
 
@@ -21,7 +21,7 @@ describe('addPlatformToRequestChain', () => {
 
   it('returns SIMORGH if the req-svc-chain header does not exist', () => {
     const headers = { 'header-name': 'header-value' };
-    const result = addPlatformToRequestChain({
+    const result = addPlatformToRequestChainHeader({
       headers,
     });
 

src/server/utilities/addPlatformToRequestChain/index.ts src/server/utilities/addPlatformToRequestChainHeader/index.ts

@@ -1,7 +1,7 @@
 /* eslint-disable import/prefer-default-export */
 import { NextResponse, NextRequest } from 'next/server';
 
-import addPlatformToRequestChain from '#server/utilities/addPlatformToRequestChain';
+import addPlatformToRequestChainHeader from '#server/utilities/addPlatformToRequestChainHeader';
 import cspHeaderResponse from './utilities/cspHeaderResponse';
 
 const LOCALHOST_DOMAINS = ['localhost', '127.0.0.1'];
@@ -22,7 +22,7 @@ export function middleware(request: NextRequest) {
 
   response.headers.set(
     'req-svc-chain',
-    addPlatformToRequestChain({
+    addPlatformToRequestChainHeader({
       headers: request.headers,
     }),
   );