That's completely unneeded.

Sent from my Windows Phone
________________________________
From: NTaylorMullen<mailto:notifications@github.com>
Sent: ý2/ý22/ý2013 10:46 AM
To: SignalR/SignalR<mailto:SignalR@noreply.github.com>
Subject: Re: [SignalR] 1556 - C# (#1565)


In src/Microsoft.AspNet.SignalR.Client/Transports/TransportHelper.cs:

> @@ -76,13 +76,9 @@ public static string GetReceiveQueryString(IConnection connection, string data,
>                  qsBuilder.Append("&connectionData=" + data);
>              }
>
> -            string customQuery = connection.QueryString;
> +            string customQuery = AppendCustomQueryString(connection, qsBuilder.ToString());


Actuallyyy, on second note, this is already done in Negotiate. This line is just to ensure that all other subsequent requests utilize the append custom query string method. It's essentially the same as it was before but with extra parameter checking (verifying how to append the query string by an ampersand or a question mark etc.)

—
Reply to this email directly or view it on GitHub<https://github.com/SignalR/SignalR/pull/1565/files#r3120346>.