Skip to content

Commit 85956a6

Browse files
fix(convert-config): don't skip certificate config if it contains duplicates (#1305)
1 parent 755923a commit 85956a6

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

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

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2765,7 +2765,12 @@ export class ConvertAseaConfig {
27652765
accountKey,
27662766
);
27672767
}
2768-
if (processedCertificates.has(certificate.name)) return;
2768+
if (processedCertificates.has(certificate.name)) {
2769+
this.configCheck.addWarning(
2770+
`Certificate ${certificate.name} found for multiple accounts (${accountKey}). Only the first configuration is converted. Verify your configuration.`,
2771+
);
2772+
continue;
2773+
}
27692774
certificates.push(await getTransformedCertificate(certificate));
27702775
processedCertificates.add(certificate.name);
27712776
}
@@ -2789,7 +2794,12 @@ export class ConvertAseaConfig {
27892794
);
27902795
}
27912796
}
2792-
if (processedCertificates.has(certificate.name)) return;
2797+
if (processedCertificates.has(certificate.name)) {
2798+
this.configCheck.addWarning(
2799+
`Certificate ${certificate.name} found for multiple OUs (${ouKey}). Only the first configuration is converted. Verify your configuration.`,
2800+
);
2801+
continue;
2802+
}
27932803
certificates.push(await getTransformedCertificate(certificate));
27942804
processedCertificates.add(certificate.name);
27952805
}

0 commit comments

Comments
 (0)