Skip to content
This repository was archived by the owner on Dec 31, 2025. It is now read-only.

Commit 5b6a377

Browse files
fix(convert-config): don't skip certificate config if it contains duplicates
1 parent ccbb287 commit 5b6a377

1 file changed

Lines changed: 12 additions & 2 deletions

File tree

reference-artifacts/Custom-Scripts/lza-upgrade/src/convert-config.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2760,7 +2760,12 @@ export class ConvertAseaConfig {
27602760
accountKey,
27612761
);
27622762
}
2763-
if (processedCertificates.has(certificate.name)) return;
2763+
if (processedCertificates.has(certificate.name)) {
2764+
this.configCheck.addWarning(
2765+
`Certificate ${certificate.name} found for multiple accounts (${accountKey}). Only the first configuration is converted. Verify your configuration.`,
2766+
);
2767+
continue;
2768+
}
27642769
certificates.push(await getTransformedCertificate(certificate));
27652770
processedCertificates.add(certificate.name);
27662771
}
@@ -2784,7 +2789,12 @@ export class ConvertAseaConfig {
27842789
);
27852790
}
27862791
}
2787-
if (processedCertificates.has(certificate.name)) return;
2792+
if (processedCertificates.has(certificate.name)) {
2793+
this.configCheck.addWarning(
2794+
`Certificate ${certificate.name} found for multiple OUs (${ouKey}). Only the first configuration is converted. Verify your configuration.`,
2795+
);
2796+
continue;
2797+
}
27882798
certificates.push(await getTransformedCertificate(certificate));
27892799
processedCertificates.add(certificate.name);
27902800
}

0 commit comments

Comments
 (0)