Create new GraphQLHttpWebSocket instance if needed

[Maydayof]

Create new GraphQLHttpWebSocket instance if the UseWebSocketForQueriesAndMutations is true

[rose-a]

Subscriptions must always use the websocket connection (obviously they don't work with regular HTTP requests).

You need to make sure that an instance of GraphQLHttpWebSocket is present when a subscription is created, otherwise you will receive a null reference exception on creating a subscription.

[rose-a]

OK, one last thing: Would you please use Lazy<GraphQLHttpWebSocket> instead of GetGraphQLHttpWebSocket()?

This has thread-safety already builtin...

[Maydayof]

To instantiate a GraphQLHttpWebSocket requires a call into an instance method and pass an instance reference (/* public GraphQLHttpWebSocket(Uri webSocketUri, GraphQLHttpClient client) */ )
But, a field initializer cannot reference non-static field, method, property. At the moment I just can't figure out, how should I use the Lazy variant? Any idea?

[rose-a]

I think its well described in the API Reference.

Create a private field for a Lazy<GraphQLHttpWebSocket> and assign it inside the constructor using the factory method () => new GraphQLHttpWebSocket(GetWebSocketUri(), this).

Then you access the actual instance of GraphQLHttpWebSocket using lazyWebsocket.Value.
The instance will then be created on fist access to lazyWebsocket.Value.

[Maydayof]

Thx, I have moved the initialization into the ctor, that was the problem.