HEAT-230 sanitise hostname (#22)

ministryofjustice · May 28, 2024 · 0c40368 · 0c40368
1 parent 5a203af
commit 0c40368
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/data/appInsights/index.test.ts b/src/data/appInsights/index.test.ts
@@ -0,0 +1,13 @@
+import { sanitize } from '.'
+
+describe('sanitise', () => {
+  test('check removes :443', () => {
+    expect(sanitize('approved-premises-api.hmpps.service.justice.gov.uk:443')).toStrictEqual('approved-premises-api.hmpps.service.justice.gov.uk')
+  })
+
+  test('check removes random suffix', () => {
+    expect(sanitize('education-employment-api.hmpps.service.justice.gov.uk | aaaaaaa-eeee-dddd-ccccc-bbbbbbbb')).toStrictEqual(
+      'education-employment-api.hmpps.service.justice.gov.uk',
+    )
+  })
+})
diff --git a/src/data/appInsights/index.ts b/src/data/appInsights/index.ts
@@ -3,11 +3,13 @@ import { type Dependency } from '../Components'
 import AppInsights from './Client'
 import Queries from './queries'
 
+export const sanitize = (s: string) => (s ? s.replace(':443', '').replace(/(.*?\.gov\.uk).*/, '$1') : '')
+
 const getDependencies = async (appInsightsCreds: AppInsightsCreds): Promise<Dependency[]> => {
   const appInsights = new AppInsights(appInsightsCreds)
   const results = await appInsights.query(Queries.DEPENDENCIES())
 
-  return results.rows.map(row => ({ componentName: row[0], dependencyHostname: row[1], type: row[2] }))
+  return results.rows.map(row => ({ componentName: row[0], dependencyHostname: sanitize(row[1]), type: row[2] }))
 }
 
 export default getDependencies