X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;ds=sidebyside;f=src%2Fcharging-station%2Focpp%2FOCPPIncomingRequestService.ts;h=4b27f9d82f8de91b38bd1a19c4a826ebe6800d6b;hb=1b2acf4e9c00cc7272ec7769b4e82113d61f64fb;hp=0c5b5c61b05651d7859cd8225edabe7d2c5ef92f;hpb=ec0eebcce06f18b41e6639b72728110bda55c250;p=e-mobility-charging-stations-simulator.git diff --git a/src/charging-station/ocpp/OCPPIncomingRequestService.ts b/src/charging-station/ocpp/OCPPIncomingRequestService.ts index 0c5b5c61..4b27f9d8 100644 --- a/src/charging-station/ocpp/OCPPIncomingRequestService.ts +++ b/src/charging-station/ocpp/OCPPIncomingRequestService.ts @@ -11,7 +11,6 @@ import type { ClearCacheResponse, HandleErrorParams, IncomingRequestCommand, - JsonObject, JsonType, OCPPVersion, } from '../../types'; @@ -23,8 +22,8 @@ export abstract class OCPPIncomingRequestService extends AsyncResource { private static instance: OCPPIncomingRequestService | null = null; private readonly version: OCPPVersion; private readonly ajv: Ajv; - private jsonValidateFunctions: Map>; - protected abstract jsonSchemas: Map>; + private jsonValidateFunctions: Map>; + protected abstract jsonSchemas: Map>; protected constructor(version: OCPPVersion) { super(moduleName); @@ -34,7 +33,7 @@ export abstract class OCPPIncomingRequestService extends AsyncResource { multipleOfPrecision: 2, }); ajvFormats(this.ajv); - this.jsonValidateFunctions = new Map>(); + this.jsonValidateFunctions = new Map>(); this.incomingRequestHandler = this.incomingRequestHandler.bind(this) as < ReqType extends JsonType, // eslint-disable-next-line @typescript-eslint/no-unused-vars @@ -90,13 +89,10 @@ export abstract class OCPPIncomingRequestService extends AsyncResource { schema: JSONSchemaType, payload: T, ): boolean { - if (chargingStation.getOcppStrictCompliance() === false) { + if (chargingStation.stationInfo?.ocppStrictCompliance === false) { return true; } - if (this.jsonValidateFunctions.has(commandName) === false) { - this.jsonValidateFunctions.set(commandName, this.ajv.compile(schema).bind(this)); - } - const validate = this.jsonValidateFunctions.get(commandName)!; + const validate = this.getJsonIncomingRequestValidateFunction(commandName, schema); if (validate(payload)) { return true; } @@ -105,7 +101,7 @@ export abstract class OCPPIncomingRequestService extends AsyncResource { validate.errors, ); throw new OCPPError( - OCPPServiceUtils.ajvErrorsToErrorType(validate.errors!), + OCPPServiceUtils.ajvErrorsToErrorType(validate.errors), 'Incoming request PDU is invalid', commandName, JSON.stringify(validate.errors, undefined, 2), @@ -119,6 +115,16 @@ export abstract class OCPPIncomingRequestService extends AsyncResource { return OCPPConstants.OCPP_RESPONSE_REJECTED; } + private getJsonIncomingRequestValidateFunction( + commandName: IncomingRequestCommand, + schema: JSONSchemaType, + ) { + if (this.jsonValidateFunctions.has(commandName) === false) { + this.jsonValidateFunctions.set(commandName, this.ajv.compile(schema).bind(this)); + } + return this.jsonValidateFunctions.get(commandName)!; + } + // eslint-disable-next-line @typescript-eslint/no-unused-vars public abstract incomingRequestHandler( chargingStation: ChargingStation,