refactor(ui): validate UUID format
authorJérôme Benoit <jerome.benoit@sap.com>
Sun, 10 Mar 2024 17:03:15 +0000 (18:03 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Sun, 10 Mar 2024 17:03:15 +0000 (18:03 +0100)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/utils/ChargingStationConfigurationUtils.ts
ui/web/src/composables/UIClient.ts
ui/web/src/composables/Utils.ts

index 0e32c7e629b28bfa3ade6eba052272130877a8ea..7b5346d52392deb1996673bc7210e1a3bf300708 100644 (file)
@@ -21,7 +21,7 @@ export const buildChargingStationAutomaticTransactionGeneratorConfiguration = (
 
 export const buildConnectorsStatus = (chargingStation: ChargingStation): ConnectorStatus[] => {
   return [...chargingStation.connectors.values()].map(
-    ({ transactionSetInterval, ...connectorStatusRest }) => connectorStatusRest
+    ({ transactionSetInterval, ...connectorStatus }) => connectorStatus
   )
 }
 
@@ -37,7 +37,7 @@ export const buildEvsesStatus = (
   // eslint-disable-next-line array-callback-return
   return [...chargingStation.evses.values()].map(evseStatus => {
     const connectorsStatus = [...evseStatus.connectors.values()].map(
-      ({ transactionSetInterval, ...connectorStatusRest }) => connectorStatusRest
+      ({ transactionSetInterval, ...connectorStatus }) => connectorStatus
     )
     let status: EvseStatusConfiguration
     switch (outputFormat) {
index c2b9d60cc84c9c41f482168f6da43ca44add44cf..571d6d5b85a64cd4bb54ef2493c0af68d5e2b0bb 100644 (file)
@@ -12,7 +12,7 @@ import {
   type UIServerConfigurationSection
 } from '@/types'
 
-import { randomUUID } from './Utils'
+import { randomUUID, validateUUID } from './Utils'
 
 type ResponseHandler = {
   procedureName: ProcedureName
@@ -240,8 +240,8 @@ export class UIClient {
     try {
       response = JSON.parse(messageEvent.data) as ProtocolResponse
     } catch (error) {
-      useToast().error('Invalid response format')
-      console.error('Invalid response format', error)
+      useToast().error('Invalid response JSON format')
+      console.error('Invalid response JSON format', error)
       return
     }
 
@@ -253,6 +253,12 @@ export class UIClient {
 
     const [uuid, responsePayload] = response
 
+    if (!validateUUID(uuid)) {
+      useToast().error('Response UUID field is invalid')
+      console.error('Response UUID field is invalid:', response)
+      return
+    }
+
     if (this.responseHandlers.has(uuid)) {
       const { procedureName, resolve, reject } = this.responseHandlers.get(uuid)!
       switch (responsePayload.status) {
index 792e4481d4b944775d2032e27ee8c7f4a758d194..0c8a83996e905c7e046d624ad9a8bd64124d1df1 100644 (file)
@@ -56,6 +56,12 @@ export const randomUUID = (): `${string}-${string}-${string}-${string}-${string}
   return crypto.randomUUID()
 }
 
+export const validateUUID = (
+  uuid: `${string}-${string}-${string}-${string}-${string}`
+): uuid is `${string}-${string}-${string}-${string}-${string}` => {
+  return /^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-4[0-9a-fA-F]{3}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$/.test(uuid)
+}
+
 export const useUIClient = (): UIClient => {
   return UIClient.getInstance()
 }