From 5b6a3779aee95cf0c43d7c1b4513fa7e66fdcafd Mon Sep 17 00:00:00 2001 From: Olivier Gaumond Date: Mon, 22 Sep 2025 16:51:51 -0400 Subject: [PATCH] fix(convert-config): don't skip certificate config if it contains duplicates --- .../lza-upgrade/src/convert-config.ts | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/reference-artifacts/Custom-Scripts/lza-upgrade/src/convert-config.ts b/reference-artifacts/Custom-Scripts/lza-upgrade/src/convert-config.ts index 39523f5ec..ad646af65 100644 --- a/reference-artifacts/Custom-Scripts/lza-upgrade/src/convert-config.ts +++ b/reference-artifacts/Custom-Scripts/lza-upgrade/src/convert-config.ts @@ -2760,7 +2760,12 @@ export class ConvertAseaConfig { accountKey, ); } - if (processedCertificates.has(certificate.name)) return; + if (processedCertificates.has(certificate.name)) { + this.configCheck.addWarning( + `Certificate ${certificate.name} found for multiple accounts (${accountKey}). Only the first configuration is converted. Verify your configuration.`, + ); + continue; + } certificates.push(await getTransformedCertificate(certificate)); processedCertificates.add(certificate.name); } @@ -2784,7 +2789,12 @@ export class ConvertAseaConfig { ); } } - if (processedCertificates.has(certificate.name)) return; + if (processedCertificates.has(certificate.name)) { + this.configCheck.addWarning( + `Certificate ${certificate.name} found for multiple OUs (${ouKey}). Only the first configuration is converted. Verify your configuration.`, + ); + continue; + } certificates.push(await getTransformedCertificate(certificate)); processedCertificates.add(certificate.name); }