Lissy93 / web-check

🕵️‍♂️ All-in-one OSINT tool for analysing any website
https://web-check.xyz
MIT License
22.47k stars 1.72k forks source link

DNS Record section titles information incorrectly #169

Open realmx2 opened 3 months ago

realmx2 commented 3 months ago

Running a check on a site at: https://web-check.as93.net/

In the "DNS Records" box - the records shown are out of order and not titled correctly.

In the example below you can see that under "CNAME" it has the Nameservers, and under NS it appears to be showint TXT records... image

berend-debug commented 2 months ago

There is an incorrect mapping of the lookup results due to the fact that the const that holds these results expects 9 results, while 10 lookups are being perfomed:

    const [a, aaaa, mx, txt, ns, cname, soa, srv, ptr] = await Promise.all([
      lookupPromise(hostname),
      resolve4Promise(hostname).catch(() => []), // A record
      resolve6Promise(hostname).catch(() => []), // AAAA record
      resolveMxPromise(hostname).catch(() => []), // MX record
      resolveTxtPromise(hostname).catch(() => []), // TXT record
      resolveNsPromise(hostname).catch(() => []), // NS record
      resolveCnamePromise(hostname).catch(() => []), // CNAME record
      resolveSoaPromise(hostname).catch(() => []), // SOA record
      resolveSrvPromise(hostname).catch(() => []), // SRV record
      resolvePtrPromise(hostname).catch(() => [])  // PTR record
    ]);

lookupPromise() should be removed, or an extra entry in to the const that accounts for the results of lookupPromise() should be added.