use Resolver class of dns/promises, avoiding messing with other network calls

Author: Soxasora

lib/domain-verification.js

@@ -1,5 +1,5 @@
 import { requestCertificate, getCertificateStatus, describeCertificate } from '@/api/acm'
-import { promises as dnsPromises } from 'node:dns'
+import { Resolver } from 'node:dns/promises'
 
 // Issue a certificate for a custom domain
 export async function issueDomainCertificate (domainName) {
@@ -69,11 +69,12 @@ export async function verifyDomainDNS (domainName, verificationTxt, verification
   }
 
   // by default use cloudflare DNS resolver
-  dnsPromises.setServers([process.env.DNS_RESOLVER || '1.1.1.1'])
+  const resolver = new Resolver()
+  resolver.setServers([process.env.DNS_RESOLVER || '1.1.1.1'])
 
   // TXT Records checking
   try {
-    const txtRecords = await dnsPromises.resolve(txtHost, 'TXT')
+    const txtRecords = await resolver.resolveTxt(txtHost)
     const txtText = txtRecords.flat().join(' ')
 
     // the TXT record should include the verificationTxt that we have in the database
@@ -88,7 +89,7 @@ export async function verifyDomainDNS (domainName, verificationTxt, verification
 
   // CNAME Records checking
   try {
-    const cnameRecords = await dnsPromises.resolve(domainName, 'CNAME')
+    const cnameRecords = await resolver.resolveCname(domainName)
 
     // the CNAME record should include the cname that we have in the database
     result.cnameValid = cnameRecords.some(record =>