Config based symlinks update

AssemblyJohn · AssemblyJohn · commit 2fadaa5a55d9 · 2024-01-24T12:44:32.000+02:00
Signed-off-by: AssemblyJohn &lt;ioan.bogdann@gmail.com&gt;
diff --git a/config/v201/component_schemas/standardized/InternalCtrlr.json b/config/v201/component_schemas/standardized/InternalCtrlr.json
@@ -471,6 +471,21 @@
           "default": "43200",
           "type": "integer"
       },
+      "UpdateCertificateSymlinks": {
+        "variable_name": "UpdateCertificateSymlinks",
+        "characteristics": {
+            "supportsMonitoring": false,
+            "dataType": "boolean"
+        },
+        "attributes": [
+            {
+                "type": "Actual",
+                "mutability": "ReadOnly"
+            }
+        ],
+        "default": "0",
+        "type": "boolean"
+      },
       "MessageQueueSizeThreshold": {
           "variable_name": "MessageQueueSizeThreshold",
           "characteristics": {
diff --git a/config/v201/config.json b/config/v201/config.json
@@ -265,7 +265,14 @@
                 "attributes": {
                     "Actual": "Enabled,Active,Available,Problem"
                 }
+            },
+            "UpdateCertificateSymlinks": {
+                "variable_name": "UpdateCertificateSymlinks",
+                "attributes": {
+                    "Actual": "0"
+                }
             }
+            
         }
     },
     {
diff --git a/include/ocpp/v201/ctrlr_component_variables.hpp b/include/ocpp/v201/ctrlr_component_variables.hpp
@@ -176,6 +176,7 @@ extern const ComponentVariable& CertSigningRepeatTimes;
 extern const ComponentVariable& CertSigningWaitMinimum;
 extern const RequiredComponentVariable& SecurityCtrlrIdentity;
 extern const ComponentVariable& MaxCertificateChainSize;
+extern const ComponentVariable& UpdateCertificateSymlinks;
 extern const RequiredComponentVariable& OrganizationName;
 extern const RequiredComponentVariable& SecurityProfile;
 extern const ComponentVariable& ACPhaseSwitchingSupported;
diff --git a/lib/ocpp/v201/charge_point.cpp b/lib/ocpp/v201/charge_point.cpp
@@ -1764,7 +1764,9 @@ void ChargePoint::handle_certificate_signed_req(Call<CertificateSignedRequest> c
     }
 
     // Trigger a symlink update for V2G certificates
-    if (cert_signing_use == ocpp::CertificateSigningUseEnum::V2GCertificate) {
+    if ((cert_signing_use == ocpp::CertificateSigningUseEnum::V2GCertificate) and
+        this->device_model->get_optional_value<bool>(ControllerComponentVariables::UpdateCertificateSymlinks)
+            .value_or(false)) {
         this->evse_security->update_certificate_links(cert_signing_use);
     }
 
diff --git a/lib/ocpp/v201/ctrlr_component_variables.cpp b/lib/ocpp/v201/ctrlr_component_variables.cpp
@@ -989,6 +989,13 @@ const ComponentVariable& MaxCertificateChainSize = {
         "MaxCertificateChainSize",
     }),
 };
+const ComponentVariable& UpdateCertificateSymlinks = {
+    ControllerComponents::InternalCtrlr,
+    std::nullopt,
+    std::optional<Variable>({
+        "UpdateCertificateSymlinks",
+    }),
+};
 const RequiredComponentVariable& OrganizationName = {
     ControllerComponents::SecurityCtrlr,
     std::nullopt,

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,14 @@`
`265`	`265`	`"attributes": {`
`266`	`266`	`"Actual": "Enabled,Active,Available,Problem"`
`267`	`267`	`}`
	`268`	`+ },`
	`269`	`+ "UpdateCertificateSymlinks": {`
	`270`	`+ "variable_name": "UpdateCertificateSymlinks",`
	`271`	`+ "attributes": {`
	`272`	`+ "Actual": "0"`
	`273`	`+ }`
`268`	`274`	`}`
	`275`	`+`
`269`	`276`	`}`
`270`	`277`	`},`
`271`	`278`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1764,7 +1764,9 @@ void ChargePoint::handle_certificate_signed_req(Call<CertificateSignedRequest> c`
`1764`	`1764`	`}`
`1765`	`1765`
`1766`	`1766`	`// Trigger a symlink update for V2G certificates`
`1767`		`- if (cert_signing_use == ocpp::CertificateSigningUseEnum::V2GCertificate) {`
	`1767`	`+ if ((cert_signing_use == ocpp::CertificateSigningUseEnum::V2GCertificate) and`
	`1768`	`+ this->device_model->get_optional_value<bool>(ControllerComponentVariables::UpdateCertificateSymlinks)`
	`1769`	`+ .value_or(false)) {`
`1768`	`1770`	`this->evse_security->update_certificate_links(cert_signing_use);`
`1769`	`1771`	`}`
`1770`	`1772`