test/internet/test-dns-idna2008.js fails #48262

Activity

1. richardlau commented on May 31, 2023

   

   richardlau

   on May 31, 2023

   Member

   More actions

   I think this started failing on the daily CI on 11 May in https://ci.nodejs.org/view/Node.js%20Daily/job/node-daily-master/3042/
2. anonrig commented on May 31, 2023

   

   anonrig

   on May 31, 2023

   Member

   More actions

   cc @nodejs/url
3. anonrig commented on May 31, 2023

   

   anonrig

   on May 31, 2023

   Last edited by anonrig

   Member

   More actions

   @ShogunPanda @targos I don't think this is related to ada-url/idna. Previously cares was returning ESERVFAIL and right now it returns ENOTFOUND, and .errno attribute is no longer returning anything.

   I've also added a test case to ada-url/idna as well.

   Here's the error message:

   {
     err: Error: queryA ENOTFOUND straße.de
         at QueryReqWrap.onresolve [as oncomplete] (node:internal/dns/callback_resolver:45:19) {
       errno: undefined,
       code: 'ENOTFOUND',
       syscall: 'queryA',
       hostname: 'straße.de'
     }
   }
   

   Here's the diff to pass the test case. I don't know if it's true.

   diff --git a/test/internet/test-dns-idna2008.js b/test/internet/test-dns-idna2008.js
   index 16c8933073..fb93b7a97a 100644
   --- a/test/internet/test-dns-idna2008.js
   +++ b/test/internet/test-dns-idna2008.js
   @@ -41,16 +41,16 @@ dns.promises.lookup(fixture.hostname, { family: fixture.family })
      .then(({ address }) => {
        assert.strictEqual(address, fixture.expectedAddress);
      }, (err) => {
   -    if (err && err.errno === 'ESERVFAIL') {
   -      assert.ok(err.message.includes('queryA ESERVFAIL straße.de'));
   +    if (err && err.code === 'ENOTFOUND') {
   +      assert.ok(err.message.includes('queryA ENOTFOUND straße.de'));
        } else {
          throw err;
        }
      }).finally(mustCall());
    
    dns.resolve4(fixture.hostname, mustCall((err, addresses) => {
   -  if (err && err.errno === 'ESERVFAIL') {
   -    assert.ok(err.message.includes('queryA ESERVFAIL straße.de'));
   +  if (err && err.code === 'ENOTFOUND') {
   +    assert.ok(err.message.includes('queryA ENOTFOUND straße.de'));
        return;
      }
      assert.ifError(err);
   @@ -61,8 +61,8 @@ const p = new dns.promises.Resolver().resolve4(fixture.hostname);
    p.then((addresses) => {
      assert.deepStrictEqual(addresses, [fixture.expectedAddress]);
    }, (err) => {
   -  if (err && err.errno === 'ESERVFAIL') {
   -    assert.ok(err.message.includes('queryA ESERVFAIL straße.de'));
   +  if (err && err.code === 'ENOTFOUND') {
   +    assert.ok(err.message.includes('queryA ENOTFOUND straße.de'));
      } else {
        throw err;
      }
4. anonrig

   added

   dnsIssues and PRs related to the dns subsystem.Issues and PRs related to the dns subsystem.

   on May 31, 2023
5. nodejs-github-bot

   closed this as completedin #48354on Jun 7, 2023