X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fcharging-station%2Focpp%2F2.0%2FOCPP20RequestService.ts;h=3e54ca50326c4bde179674662b50bfbe5ed6d6bb;hb=81533a206ec56709897f27edf1298e7c86d74c31;hp=e23b1e76fcc3e1fbf9b64f2e61f3d44f0fdb98d2;hpb=96a52d088b4927e7bc51ea9282f502c2a6b67707;p=e-mobility-charging-stations-simulator.git diff --git a/src/charging-station/ocpp/2.0/OCPP20RequestService.ts b/src/charging-station/ocpp/2.0/OCPP20RequestService.ts index e23b1e76..3e54ca50 100644 --- a/src/charging-station/ocpp/2.0/OCPP20RequestService.ts +++ b/src/charging-station/ocpp/2.0/OCPP20RequestService.ts @@ -10,12 +10,12 @@ import OCPPError from '../../../exception/OCPPError'; import type { JsonObject, JsonType } from '../../../types/JsonType'; import { type OCPP20BootNotificationRequest, + type OCPP20HeartbeatRequest, OCPP20RequestCommand, } from '../../../types/ocpp/2.0/Requests'; import { ErrorType } from '../../../types/ocpp/ErrorType'; import { OCPPVersion } from '../../../types/ocpp/OCPPVersion'; import type { RequestParams } from '../../../types/ocpp/Requests'; -import logger from '../../../utils/Logger'; import Utils from '../../../utils/Utils'; import type ChargingStation from '../../ChargingStation'; import OCPPRequestService from '../OCPPRequestService'; @@ -25,7 +25,7 @@ import { OCPP20ServiceUtils } from './OCPP20ServiceUtils'; const moduleName = 'OCPP20RequestService'; export default class OCPP20RequestService extends OCPPRequestService { - private jsonSchemas: Map>; + protected jsonSchemas: Map>; public constructor(ocppResponseService: OCPPResponseService) { if (new.target?.name === moduleName) { @@ -45,9 +45,20 @@ export default class OCPP20RequestService extends OCPPRequestService { ) ) as JSONSchemaType, ], + [ + OCPP20RequestCommand.HEARTBEAT, + JSON.parse( + fs.readFileSync( + path.resolve( + path.dirname(fileURLToPath(import.meta.url)), + '../../../assets/json-schemas/ocpp/2.0/HeartbeatRequest.json' + ), + 'utf8' + ) + ) as JSONSchemaType, + ], ]); this.buildRequestPayload.bind(this); - this.validatePayload.bind(this); } public async requestHandler( @@ -62,7 +73,6 @@ export default class OCPP20RequestService extends OCPPRequestService { commandName, commandParams ); - this.validatePayload(chargingStation, commandName, requestPayload); return (await this.sendMessage( chargingStation, Utils.generateUUID(), @@ -88,28 +98,33 @@ export default class OCPP20RequestService extends OCPPRequestService { commandParams = commandParams as JsonObject; switch (commandName) { case OCPP20RequestCommand.BOOT_NOTIFICATION: - commandParams.modem = commandParams.modem as JsonObject; + commandParams.chargingStation = commandParams.chargingStation as JsonObject; + commandParams.chargingStation.modem = commandParams.chargingStation.modem as JsonObject; return { reason: commandParams?.reason, chargingStation: { - model: commandParams?.model, - vendorName: commandParams?.vendorName, - ...(!Utils.isUndefined(commandParams?.firmwareVersion) && { - firmwareVersion: commandParams.firmwareVersion, + model: commandParams?.chargingStation?.model, + vendorName: commandParams?.chargingStation?.vendorName, + ...(!Utils.isUndefined(commandParams?.chargingStation?.firmwareVersion) && { + firmwareVersion: commandParams.chargingStation?.firmwareVersion, + }), + ...(!Utils.isUndefined(commandParams?.chargingStation?.serialNumber) && { + serialNumber: commandParams.chargingStation?.serialNumber, }), - ...(!Utils.isUndefined(commandParams?.serialNumber) && { - serialNumber: commandParams.serialNumber, + ...(!Utils.isUndefined(commandParams?.chargingStation?.modem) && { + modem: { + ...(!Utils.isUndefined(commandParams?.chargingStation?.modem?.iccid) && { + iccid: commandParams.chargingStation.modem.iccid, + }), + ...(!Utils.isUndefined(commandParams?.chargingStation?.modem?.imsi) && { + imsi: commandParams.chargingStation.modem.imsi, + }), + }, }), - modem: { - ...(!Utils.isUndefined(commandParams?.modem?.iccid) && { - iccid: commandParams.modem.iccid, - }), - ...(!Utils.isUndefined(commandParams?.modem?.imsi) && { - imsi: commandParams.modem.imsi, - }), - }, }, } as unknown as Request; + case OCPP20RequestCommand.HEARTBEAT: + return {} as unknown as Request; default: // OCPPError usage here is debatable: it's an error in the OCPP stack but not targeted to sendError(). throw new OCPPError( @@ -121,23 +136,4 @@ export default class OCPP20RequestService extends OCPPRequestService { ); } } - - private validatePayload( - chargingStation: ChargingStation, - commandName: OCPP20RequestCommand, - requestPayload: Request - ): boolean { - if (this.jsonSchemas.has(commandName)) { - return this.validateRequestPayload( - chargingStation, - commandName, - this.jsonSchemas.get(commandName), - requestPayload - ); - } - logger.warn( - `${chargingStation.logPrefix()} ${moduleName}.validatePayload: No JSON schema found for command ${commandName} PDU validation` - ); - return false; - } }