Sec-WebSocket-Protocol is aggregated wrong

[dietert]

Hello,

Sec-WebSocket-Protocol is aggregated wrong

See:

WebsocketClientLite.PCL/src/main/WebsocketClientLite.Netstandard/Helper/ClientHandShake.cs

Line 43 in 119be3d

.Aggregate("Sec-WebSocket-Protocol: ", (current, protocol) => $"{current}, {protocol}");

E.g: following line

List subprotocols = new List {"soap", "json"};

Will insert into the headers (wrong)

Sec-WebSocket-Protocol: , soap, json

Instead of (correct)

Sec-WebSocket-Protocol: soap, json

Possible solution:

var subprotocol = $"Sec-WebSocket-Protocol: {subprotocols.Aggregate((current, protocol) => $"{current}, {protocol}")}";

[1iveowl]

Good catch. It's been fixed. in version 4.14.