BAU: Remove sourceIp from config options for request config function

This is no longer used since we switched to using the common headers library to set the x-forwarded-for header on requests to the backend, so can be removed
govuk-one-login · May 22, 2024 · 65ffcca · 65ffcca
1 parent 08b60cd
commit 65ffcca
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/src/utils/http.ts b/src/utils/http.ts
@@ -26,7 +26,6 @@ export interface ConfigOptions {
   sessionId?: string;
   clientSessionId?: string;
   validationStatuses?: number[];
-  sourceIp?: string;
   persistentSessionId?: string;
   baseURL?: string;
   userLanguage?: string;

diff --git a/test/unit/utils/http.test.ts b/test/unit/utils/http.test.ts
@@ -47,7 +47,6 @@ describe("getInternalRequestConfigWithSecurityHeaders", () => {
       const actualConfig = getInternalRequestConfigWithSecurityHeaders(
         {
           sessionId: sessionId,
-          sourceIp: sourceIp,
           clientSessionId: clientSessionId,
           persistentSessionId: persistentSessionId,
           reauthenticate: reauthenticate,
@@ -78,13 +77,11 @@ describe("getInternalRequestConfigWithSecurityHeaders", () => {
       };
 
       const sessionId = "someSessionId";
-      const sourceIp = "123.123.123.123";
       const clientSessionId = "someClientSessionId";
       const persistentSessionId = "somePersistentSessionId";
       const actualConfig = getInternalRequestConfigWithSecurityHeaders(
         {
           sessionId: sessionId,
-          sourceIp: sourceIp,
           clientSessionId: clientSessionId,
           persistentSessionId: persistentSessionId,
         },