Friendly fixes for RSocketClient.ConnectAsync() - since this is a low-frequency call, no harm in taking the byte[] primitive.

Author: dbarnett-netifi

RSocket.Core/RSocketClient.cs

@@ -20,11 +20,10 @@ public class RSocketClient : RSocket
 
 		public RSocketClient(IRSocketTransport transport, RSocketOptions options = default) : base(transport, options) { }
 
-		public Task ConnectAsync(RSocketOptions options = default, byte[] data = default, byte[] metadata = default) => ConnectAsync(options, data: data == default ? default : new ReadOnlySequence<byte>(data), metadata: metadata == default ? default : new ReadOnlySequence<byte>(metadata));
+		public Task ConnectAsync(RSocketOptions options = default, byte[] data = default, byte[] metadata = default) => ConnectAsync(options ?? RSocketOptions.Default, data: data == default ? default : new ReadOnlySequence<byte>(data), metadata: metadata == default ? default : new ReadOnlySequence<byte>(metadata));
 
-		public async Task ConnectAsync(RSocketOptions options = default, ReadOnlySequence<byte> metadata = default, ReadOnlySequence<byte> data = default)
+		async Task ConnectAsync(RSocketOptions options, ReadOnlySequence<byte> metadata, ReadOnlySequence<byte> data)
 		{
-			options = options ?? RSocketOptions.Default;
 			await Transport.StartAsync();
 			Handler = Connect(CancellationToken.None);
 			await Setup(options.KeepAlive, options.Lifetime, options.MetadataMimeType, options.DataMimeType, data: data, metadata: metadata);