Merge pull request #97 from nullinside-development-group/feat/update

ProgrammingByPermutation · web-flow · commit 4036effc1e17 · 2025-06-12T16:48:46.000-04:00
fix: unhandle exception throwing when disconnecting
diff --git a/src/Nullinside.Api.Common/Twitch/TwitchClientProxy.cs b/src/Nullinside.Api.Common/Twitch/TwitchClientProxy.cs
@@ -117,8 +117,11 @@ public string? TwitchOAuthToken {
       // If we have a client, try to connect.
       if (null != _client) {
         _client.Disconnect();
-        _client.SetConnectionCredentials(new ConnectionCredentials(TwitchUsername, value));
-        Connect();
+
+        if (null != value) {
+          _client.SetConnectionCredentials(new ConnectionCredentials(TwitchUsername, value));
+          Connect();
+        }
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -117,8 +117,11 @@ public string? TwitchOAuthToken {`
`117`	`117`	`// If we have a client, try to connect.`
`118`	`118`	`if (null != _client) {`
`119`	`119`	`_client.Disconnect();`
`120`		`- _client.SetConnectionCredentials(new ConnectionCredentials(TwitchUsername, value));`
`121`		`- Connect();`
	`120`	`+`
	`121`	`+ if (null != value) {`
	`122`	`+ _client.SetConnectionCredentials(new ConnectionCredentials(TwitchUsername, value));`
	`123`	`+ Connect();`
	`124`	`+ }`
`122`	`125`	`}`
`123`	`126`	`}`
`124`	`127`	`}`