From 1edb5e66721145c567fa31babb68604bed5c69b4 Mon Sep 17 00:00:00 2001 From: tstephen-nhs <231503406+tstephen-nhs@users.noreply.github.com> Date: Mon, 30 Mar 2026 11:16:18 +0000 Subject: [PATCH 1/2] fix: log proxygen style app id and name --- .../src/getMyPrescriptions.ts | 23 ++++++++----------- .../getMyPrescriptions/src/statusUpdate.ts | 3 +-- 2 files changed, 10 insertions(+), 16 deletions(-) diff --git a/packages/getMyPrescriptions/src/getMyPrescriptions.ts b/packages/getMyPrescriptions/src/getMyPrescriptions.ts index 13ca3d562..494d753f7 100644 --- a/packages/getMyPrescriptions/src/getMyPrescriptions.ts +++ b/packages/getMyPrescriptions/src/getMyPrescriptions.ts @@ -83,7 +83,6 @@ async function eventHandler( ): Promise { const traceIDs: TraceIDs = logTraceIds(headers) const spineClient = params.spineClient - const applicationName = headers["nhsd-application-name"] ?? "unknown" const correlationId = headers["nhsd-correlation-id"] ?? crypto.randomUUID() checkSpineCertificateConfiguration(spineClient) @@ -93,7 +92,7 @@ async function eventHandler( const response = await makeSpinePrescriptionCall(spineClient, headers, params) const searchsetBundle: Bundle = response.data - logPrescriptionResponse(searchsetBundle, traceIDs, headers, applicationName) + logPrescriptionResponse(searchsetBundle, traceIDs, headers) const statusUpdateData = includeStatusUpdateData ? buildStatusUpdateData(logger, searchsetBundle) : undefined @@ -130,8 +129,7 @@ function logTraceIds(headers: EventHeaders) { return traceIDs } -function logPrescriptionResponse(searchsetBundle: Bundle, - traceIDs: TraceIDs, headers: EventHeaders, applicationName: string) { +function logPrescriptionResponse(searchsetBundle: Bundle, traceIDs: TraceIDs, headers: EventHeaders) { searchsetBundle.id = traceIDs["x-request-id"] || "unknown" const operationOutcomes = isolateOperationOutcome(searchsetBundle) @@ -140,16 +138,13 @@ function logPrescriptionResponse(searchsetBundle: Bundle, }) const ODSCodes = extractOdsCodes(logger, searchsetBundle) - logger.info( - "Processing PfP get prescriptions request for patient. " - + "They have these relevant ODS codes, and the PfP request was made via this apigee application.", - { - ODSCodes, - actorNhsNumber: headers[NHS_LOGIN_HEADER], - subjectNhsNumber: headers["nhsNumber"], - applicationName - } - ) + logger.info("Get prescriptions for patient response.", { + ODSCodes, + actorNhsNumber: headers[NHS_LOGIN_HEADER], + subjectNhsNumber: headers["nhsNumber"], + applicationName: headers["nhsd-application-name"] ?? "unknown", + applicationId: headers["nhsd-application-id"] ?? "unknown" + }) } async function makeSpinePrescriptionCall(spineClient: SpineClient, headers: EventHeaders, params: HandlerParams) { diff --git a/packages/getMyPrescriptions/src/statusUpdate.ts b/packages/getMyPrescriptions/src/statusUpdate.ts index cefd93408..990d80a37 100644 --- a/packages/getMyPrescriptions/src/statusUpdate.ts +++ b/packages/getMyPrescriptions/src/statusUpdate.ts @@ -31,8 +31,7 @@ export function buildStatusUpdateData(logger: Logger, searchsetBundle: Bundle): logger.info(`Building status update data for prescription ${prescriptionID}.`) if (allItemsApprovedOrCancelled) { - logger.info(`All items for prescription ${prescriptionID} are 'Prescriber Approved' or ' Prescriber Cancelled'.`) - logger.info(`Ignoring prescription.`) + logger.info(`No update of prescription ${prescriptionID} as all 'Prescriber Approved' or 'Prescriber Cancelled'.`) return } From 4a0adc437f574b347bdfd95d2c8d8a3588962740 Mon Sep 17 00:00:00 2001 From: tstephen-nhs <231503406+tstephen-nhs@users.noreply.github.com> Date: Mon, 30 Mar 2026 11:24:54 +0000 Subject: [PATCH 2/2] fix: include proxygen name & id --- packages/getMyPrescriptions/src/getMyPrescriptions.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/getMyPrescriptions/src/getMyPrescriptions.ts b/packages/getMyPrescriptions/src/getMyPrescriptions.ts index 494d753f7..2b538c5dc 100644 --- a/packages/getMyPrescriptions/src/getMyPrescriptions.ts +++ b/packages/getMyPrescriptions/src/getMyPrescriptions.ts @@ -143,7 +143,9 @@ function logPrescriptionResponse(searchsetBundle: Bundle, traceIDs: TraceIDs, he actorNhsNumber: headers[NHS_LOGIN_HEADER], subjectNhsNumber: headers["nhsNumber"], applicationName: headers["nhsd-application-name"] ?? "unknown", - applicationId: headers["nhsd-application-id"] ?? "unknown" + applicationId: headers["nhsd-application-id"] ?? "unknown", + proxygenAppName: headers["developer.app.name"] ?? "unknown", + proxygenAppId: headers["developer.app.id"] ?? "unknown" }) }