Merge pull request #653 from DevelopingSpace/main

fix: Account for ACME order deduplication
DevelopingSpace · Apr 20, 2023 · 013a6a9 · 013a6a9
2 parents 0516f72 + cf43254
commit 013a6a9
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/app/queues/certificate/certificate-error-handler.server.ts b/app/queues/certificate/certificate-error-handler.server.ts
@@ -57,6 +57,12 @@ export const initCertificateErrorHandler = () => {
 
  await certificateModel.updateCertificateById(certificateId, {
  status: CertificateStatus.failed,
+ /**
+ * Setting orderUrl to null, so if we get back the same cert request from the
+ * ACME provider next time (as it deduplicates orders) we are not violating the
+ * unique constraint of this field
+ */
+ orderUrl: null,
  });
 
  await challengeModel.deleteChallengesByCertificateId(certificateId);