concurrent requests cause StreamIDTooSmall error about swift-nio-http2

[Lukasa]

nextOutboundStreamID is definitely thread-safe, as we guard it by an eventLoop.execute which forces a swap onto the event loop thread.

The issue here is actually long-standing:swift-nio-http2 allocates stream IDs before it has actually created the stream. It does this for a number of reasons, but it is fundamentally brittle. The result is that if you are calling createStreamChannel repeatedly and quickly you need to ensure that you actually write data on those channels in the order that you created them, or you'll see this error, where the stream channel created second sends its data first.

There are a bunch of things we can do to fix this, but all of them are quite substantial API changes. The ultimate behaviour has to be to create a new HTTP stream channel that does not expect its users to know the stream ID (that is, where the data being passed in the pipeline is HTTP2Frame.Payload, not HTTP2Frame). Then we can lazily allocate the stream ID as and when the channel sends its first payload data.

This is not the kind of thing we'd be able to fix quickly, though, so we should also investigate a quick workaround in grpc.

[i9]

Thanks for the prompt response! will close this and continue discussions in grpc-swift repo.

[Lukasa]

Ok, tracking the underlying issue in #214.