Send notification only once

Signed-off-by: Marc Emmers <m.emmers@alfen.com>

Author: marcemmers

include/ocpp/v201/charge_point.hpp

@@ -214,6 +214,7 @@ class ChargePoint : ocpp::ChargingStationBase {
     BootReasonEnum bootreason;
     int network_configuration_priority;
     bool disable_automatic_websocket_reconnects;
+    bool skip_invalid_csms_certificate_notifications;
 
     /// \brief Component responsible for maintaining and persisting the operational status of CS, EVSEs, and connectors.
     std::shared_ptr<ComponentStateManager> component_state_manager;

lib/ocpp/v201/charge_point.cpp

@@ -59,6 +59,7 @@ ChargePoint::ChargePoint(const std::map<int32_t, int32_t>& evse_connector_struct
     registration_status(RegistrationStatusEnum::Rejected),
     network_configuration_priority(0),
     disable_automatic_websocket_reconnects(false),
+    skip_invalid_csms_certificate_notifications(false),
     reset_scheduled(false),
     reset_scheduled_evseids{},
     firmware_status(FirmwareStatusEnum::Idle),
@@ -713,6 +714,9 @@ void ChargePoint::init_websocket() {
             this->init_certificate_expiration_check_timers(); // re-init as timers are stopped on disconnect
         }
         this->time_disconnected = std::chrono::time_point<std::chrono::steady_clock>();
+
+        // We have a connection again so next time it fails we should send the notification again
+        this->skip_invalid_csms_certificate_notifications = false;
     });
 
     this->websocket->register_disconnected_callback([this]() {
@@ -749,8 +753,13 @@ void ChargePoint::init_websocket() {
     this->websocket->register_connection_failed_callback([this](ConnectionFailedReason reason) {
         switch (reason) {
         case ConnectionFailedReason::InvalidCSMSCertificate:
-            this->security_event_notification_req(CiString<50>(ocpp::security_events::INVALIDCSMSCERTIFICATE),
-                                                  std::nullopt, true, true);
+            if (!this->skip_invalid_csms_certificate_notifications) {
+                this->security_event_notification_req(CiString<50>(ocpp::security_events::INVALIDCSMSCERTIFICATE),
+                                                      std::nullopt, true, true);
+                this->skip_invalid_csms_certificate_notifications = true;
+            } else {
+                EVLOG_info << "Skipping InvalidCsmsCertificate SecurityEvent since it has been sent already";
+            }
             break;
         }
     });