Skip to content

Commit c4fcd3f

Browse files
committed
CCM-12744: case-insensitive headers
1 parent 5ff2592 commit c4fcd3f

File tree

16 files changed

+47
-22
lines changed

16 files changed

+47
-22
lines changed

lambdas/backend-api/src/__tests__/api/delete-routing-config.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,7 @@ describe('Delete Routing Config Handler', () => {
170170
authorizer: { user: 'sub', clientId: 'nhs-notify-client-id' },
171171
},
172172
pathParameters: { routingConfigId: '1-2-3' },
173+
headers: {},
173174
});
174175

175176
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/delete.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ describe('Template API - Delete', () => {
166166
authorizer: { user: 'sub', clientId: 'nhs-notify-client-id' },
167167
},
168168
pathParameters: { templateId: '1-2-3' },
169+
headers: {},
169170
});
170171

171172
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/proof.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ describe('Template API - request proof', () => {
123123
authorizer: { user: 'sub', clientId: 'nhs-notify-client-id' },
124124
},
125125
pathParameters: { templateId: 'template-id' },
126+
headers: {},
126127
});
127128

128129
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/submit-routing-config.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,7 @@ describe('Submit Routing Config Handler', () => {
174174
authorizer: { user: 'sub', clientId: 'nhs-notify-client-id' },
175175
},
176176
pathParameters: { routingConfigId: '1-2-3' },
177+
headers: {},
177178
});
178179

179180
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/submit.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,7 @@ describe('Template API - Submit', () => {
129129
authorizer: { user: 'sub', clientId: 'nhs-notify-client-id' },
130130
},
131131
pathParameters: { templateId: '1-2-3' },
132+
headers: {},
132133
});
133134

134135
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/update-routing-config.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -258,6 +258,7 @@ describe('Update Routing Config Handler', () => {
258258
},
259259
body: JSON.stringify(update),
260260
pathParameters: { routingConfigId: '1-2-3' },
261+
headers: {},
261262
});
262263

263264
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/__tests__/api/update.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,7 @@ describe('Template API - Update', () => {
261261
},
262262
body: JSON.stringify(update),
263263
pathParameters: { templateId: '1-2-3' },
264+
headers: {},
264265
});
265266

266267
const result = await handler(event, mock<Context>(), jest.fn());

lambdas/backend-api/src/api/delete-routing-config.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
import type { APIGatewayProxyHandler } from 'aws-lambda';
2-
import { apiFailure, noContent } from './responses';
3-
import type { RoutingConfigClient } from '../app/routing-config-client';
42
import { logger } from 'nhs-notify-web-template-management-utils/logger';
3+
import { apiFailure, noContent } from '@backend-api/api/responses';
4+
import type { RoutingConfigClient } from '@backend-api/app/routing-config-client';
5+
import { toHeaders } from '@backend-api/utils/headers';
56

67
export function createHandler({
78
routingConfigClient,
@@ -23,7 +24,7 @@ export function createHandler({
2324
const { error } = await routingConfigClient.deleteRoutingConfig(
2425
routingConfigId,
2526
user,
26-
event.headers['X-Lock-Number'] ?? ''
27+
toHeaders(event.headers).get('X-Lock-Number') ?? ''
2728
);
2829

2930
if (error) {

lambdas/backend-api/src/api/delete.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import type { APIGatewayProxyHandler } from 'aws-lambda';
2-
import { apiFailure, noContent } from './responses';
3-
import { TemplateClient } from '../app/template-client';
2+
import { apiFailure, noContent } from '@backend-api/api/responses';
3+
import type { TemplateClient } from '@backend-api/app/template-client';
4+
import { toHeaders } from '@backend-api/utils/headers';
45

56
export function createHandler({
67
templateClient,
@@ -22,7 +23,7 @@ export function createHandler({
2223
userId,
2324
clientId,
2425
},
25-
event.headers['X-Lock-Number'] ?? ''
26+
toHeaders(event.headers).get('X-Lock-Number') ?? ''
2627
);
2728

2829
if (error) {

lambdas/backend-api/src/api/proof.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import type { APIGatewayProxyHandler } from 'aws-lambda';
2-
import { apiFailure, apiSuccess } from './responses';
3-
import { TemplateClient } from '../app/template-client';
2+
import { apiFailure, apiSuccess } from '@backend-api/api/responses';
3+
import { TemplateClient } from '@backend-api/app/template-client';
4+
import { toHeaders } from '@backend-api/utils/headers';
45

56
export function createHandler({
67
templateClient,
@@ -22,7 +23,7 @@ export function createHandler({
2223
userId,
2324
clientId,
2425
},
25-
event.headers['X-Lock-Number'] ?? ''
26+
toHeaders(event.headers).get('X-Lock-Number') ?? ''
2627
);
2728

2829
if (error) {

0 commit comments

Comments
 (0)