From dfda9162839842eb56e80f0c528d045e32a35b91 Mon Sep 17 00:00:00 2001
From: Charles Quesnel <quesnel.charles@gmail.com>
Date: Thu, 13 Dec 2018 14:54:31 -0500
Subject: [PATCH] Use default parameter instead of overload

---
 .../Client/MqttClientOptionsBuilder.cs        | 21 +++----------------
 1 file changed, 3 insertions(+), 18 deletions(-)

diff --git a/Source/MQTTnet/Client/MqttClientOptionsBuilder.cs b/Source/MQTTnet/Client/MqttClientOptionsBuilder.cs
index e612981b..aaf4f146 100644
--- a/Source/MQTTnet/Client/MqttClientOptionsBuilder.cs
+++ b/Source/MQTTnet/Client/MqttClientOptionsBuilder.cs
@@ -92,28 +92,13 @@ namespace MQTTnet.Client
             return this;
         }
 
-        public MqttClientOptionsBuilder WithWebSocketServer(string uri)
+        public MqttClientOptionsBuilder WithWebSocketServer(string uri, MqttClientOptionsBuilderWebSocketParameters parameters = null)
         {
-            _webSocketOptions = new MqttClientWebSocketOptions
-            {
-                Uri = uri
-            };
-
-            return this;
-        }
-
-        public MqttClientOptionsBuilder WithWebSocketServer(string uri, MqttClientOptionsBuilderWebSocketParameters parameters)
-        {
-            if(parameters == null)
-            {
-                throw new ArgumentNullException(nameof(parameters));
-            }
-
             _webSocketOptions = new MqttClientWebSocketOptions
             {
                 Uri = uri,
-                RequestHeaders = parameters.RequestHeaders,
-                CookieContainer = parameters.CookieContainer
+                RequestHeaders = parameters?.RequestHeaders,
+                CookieContainer = parameters?.CookieContainer
             };
 
             return this;