dns: make promise API fully constructed from lib/internal/dns/promises

PR-URL: #43227 Refs: https://github.com/nodejs/node/discussions/43198 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>

Author: F3n67u

lib/dns.js

@@ -439,8 +439,6 @@ ObjectDefineProperties(module.exports,{
 get(){
 if(promises===null){
 promises=require('internal/dns/promises');
-promises.setServers=defaultResolverSetServers;
-promises.setDefaultResultOrder=setDefaultResultOrder;
 }
 returnpromises;
 }

lib/dns/promises.js

@@ -1,5 +1,3 @@
 'use strict';
 
-constdnsPromises=require('internal/dns/promises');
-dnsPromises.setServers=require('dns').setServers;
-module.exports=dnsPromises;
+module.exports=require('internal/dns/promises');

lib/internal/dns/promises.js

@@ -17,6 +17,8 @@ const{
  emitInvalidHostnameWarning,
  getDefaultVerbatim,
 errorCodes: dnsErrorCodes,
+ setDefaultResultOrder,
+ setDefaultResolver,
 }=require('internal/dns/utils');
 const{
 NODATA,
@@ -349,11 +351,20 @@ Resolver.prototype.resolve = function resolve(hostname, rrtype){
 returnReflectApply(resolver,this,[hostname]);
 };
 
+functiondefaultResolverSetServers(servers){
+constresolver=newResolver();
+
+resolver.setServers(servers);
+setDefaultResolver(resolver);
+bindDefaultResolver(module.exports,Resolver.prototype);
+}
 
 module.exports={
  lookup,
  lookupService,
  Resolver,
+ setDefaultResultOrder,
+setServers: defaultResolverSetServers,
 
 // ERROR CODES
 NODATA,