No pude hacer que mi ciclo for se ejecutara sincrónicamente. El orden de registro que devuelve para cada dominio cambia. Supongo que el que responda más rápido traerá esos registros primero. ¿Dónde estoy haciendo mal?
const rrtypes= ["A","MX","CNAME","NS","TXT"]; var resData = []; export const getAllRecords = async (req,res) => { const {domain} = req.params; for await(const rrtype of rrtypes){ dns.resolve (domain, rrtype, (err, records) => { resData.push(rrtype+" "+records); }); } res.send(resData); resData = []; }Tenga en cuenta que puede usar la función dns.promises.resolve : https://nodejs.org/api/dns.html#dns_dnspromises_resolve_hostname_rrtype
entonces cambiarías tu código a:
const rrtypes = ["A", "MX", "CNAME", "NS", "TXT"]; export const getAllRecords = async (req, res) => { const { domain } = req.params; // Notice that resData should be local to this function let resData = []; for (const rrtype of rrtypes) { try { const records = await dns.promises.resolve(domain, rrtype); resData.push(rrtype + " " + records); // IMO this would be better: resData.push({ type: rrtype, value: records }); } catch (err) { // Log the error here } } res.send(resData); };El problema radica en su uso de await . await debe usarse al llamar a funciones asíncronas. En este momento lo está utilizando al crear instancias de variables de una matriz.
Gracias a un comentario de slebetman, pude ver que dns.resolve no devuelve una promesa, sino que usa devoluciones de llamada para administrar las llamadas asíncronas. Como también sugirió, podemos solucionar esto creando una promesa para administrar estas devoluciones de llamada.
const rrtypes= ["A","MX","CNAME","NS","TXT"]; var resData = []; const dnsResolvePromise = (domain, rrtype) => { return new Promise((resolve, reject) => { dns.resolve(domain, rrtype, (err, records) => { if(err) return reject(err); resolve(rrtype+" "+records); }); }) } export const getAllRecords = async (req,res) => { const {domain} = req.params; for(const rrtype of rrtypes){ try{ const records = await dnsResolvePromise(domain, rrtype); resData.push(records); } catch (e){ // Handle error } } res.send(resData); resData = []; }