Merge pull request #727 from SQKo/patch-54 build docs

Make default dnsConfig

docs/src/pages/api/02_basics.md

@@ -89,7 +89,7 @@ Useful if you want to use other React components. By default, a new loop is crea
     'logger' => new \Monolog\Logger('New logger'),
 ```
 
-`dnsConfig` is an instace of `Config` or a string of name server address. By default, 8.8.8.8 is used. Currently only used for VoiceClient.
+`dnsConfig` is an instace of `Config` or a string of name server address. By default system setting is used and fall back to 8.8.8.8 when system configuration is not found. Currently only used for VoiceClient.
 
 ```php
     'dnsConfig' => '1.1.1.1',

src/Discord/Discord.php

@@ -1312,7 +1312,6 @@ protected function resolveOptions(array $options = []): array
                 'retrieveBans' => false,
                 'intents' => Intents::getDefaultIntents(),
                 'socket_options' => [],
-                'dnsConfig' => '8.8.8.8'
             ])
             ->setAllowedTypes('token', 'string')
             ->setAllowedTypes('logger', ['null', LoggerInterface::class])
@@ -1334,6 +1333,15 @@ protected function resolveOptions(array $options = []): array
             $options['logger'] = $logger;
         }
 
+        if (! isset($options['dnsConfig'])) {
+            $dnsConfig = \React\Dns\Config\Config::loadSystemConfigBlocking();
+            if (! $dnsConfig->nameservers) {
+                $dnsConfig->nameservers[] = '8.8.8.8';
+            }
+
+            $options['dnsConfig'] = $dnsConfig;
+        }
+
         if (is_array($options['intents'])) {
             $intent = 0;
             $validIntents = Intents::getValidIntents();

src/Discord/Voice/VoiceClient.php

@@ -321,7 +321,7 @@ class VoiceClient extends EventEmitter
     /**
      * The Config for DNS Resolver
      *
-     * @var string|\React\Dns\Config
+     * @var string|\React\Dns\Config\Config
      */
     protected $dnsConfig;