From e0b3812177e63f0f9e7bb52eec6d9eda7fe8a095 Mon Sep 17 00:00:00 2001 From: Jamie B <53781962+JamieB-gu@users.noreply.github.com> Date: Tue, 12 Aug 2025 13:59:00 +0100 Subject: [PATCH 1/2] Remove GET endpoints from PROD server These are not, and should not, be used. The PROD server uses the POST endpoints above. --- dotcom-rendering/src/server/server.prod.ts | 61 ---------------------- 1 file changed, 61 deletions(-) diff --git a/dotcom-rendering/src/server/server.prod.ts b/dotcom-rendering/src/server/server.prod.ts index 9f97f468624..8693728b794 100644 --- a/dotcom-rendering/src/server/server.prod.ts +++ b/dotcom-rendering/src/server/server.prod.ts @@ -89,67 +89,6 @@ export const prodServer = (): void => { app.post('/AppsBlocks', logRenderTime, handleAppsBlocks); app.post('/EditionsCrossword', logRenderTime, handleEditionsCrossword); - // These GET's are for checking any given URL directly from PROD - app.get( - '/Article/*url', - logRenderTime, - getContentFromURLMiddleware, - handleArticle, - ); - - app.get( - '/AMPArticle/*url', - logRenderTime, - getContentFromURLMiddleware, - handleAMPArticle, - ); - - app.get( - '/Front/*url', - logRenderTime, - getContentFromURLMiddleware, - handleFront, - ); - - app.get( - '/TagPage/*url', - logRenderTime, - getContentFromURLMiddleware, - handleTagPage, - ); - - app.get( - '/EmailNewsletters', - logRenderTime, - getContentFromURLMiddleware, - handleAllEditorialNewslettersPage, - ); - - app.get( - '/AppsArticle/*url', - logRenderTime, - getContentFromURLMiddleware, - handleAppsArticle, - ); - - app.get( - '/AppsInteractive/*url', - logRenderTime, - getContentFromURLMiddleware, - handleAppsInteractive, - ); - - app.get('/', (req, res) => { - res.send(` - - - -

👋

- - - `); - }); - // All params to error handlers must be declared for express to identify them as error middleware // https://expressjs.com/en/api.html#:~:text=Error%2Dhandling%20middleware%20always,see%3A%20Error%20handling // eslint-disable-next-line @typescript-eslint/no-unused-vars -- all params to error handlers must be declared From 91995a2306b329159244f3037015ec68840ebb55 Mon Sep 17 00:00:00 2001 From: Jamie B <53781962+JamieB-gu@users.noreply.github.com> Date: Tue, 12 Aug 2025 14:00:32 +0100 Subject: [PATCH 2/2] Remove unused import --- dotcom-rendering/src/server/server.prod.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/dotcom-rendering/src/server/server.prod.ts b/dotcom-rendering/src/server/server.prod.ts index 8693728b794..99e98d850c6 100644 --- a/dotcom-rendering/src/server/server.prod.ts +++ b/dotcom-rendering/src/server/server.prod.ts @@ -24,7 +24,6 @@ import { handleFootballTablesPage, } from './handler.sportDataPage.web'; import { recordBaselineCloudWatchMetrics } from './lib/aws/metrics-baseline'; -import { getContentFromURLMiddleware } from './lib/get-content-from-url'; import { logger } from './lib/logging'; import { requestLoggerMiddleware } from './lib/logging-middleware'; import { recordError } from './lib/logging-store';