Benoit Foucher

Review what we should call from the Slic read frames loop

1

> > 7\. I am pretty sure this ReadFrameAsync method executes in the critical "read loop" where we need to be really careful about not doing too much work >...

Slow Quic tests on Linux

4

On Linux, the Quic tests take more than 30s to complete. They used to complete much faster and they still do on Windows. I'm seeing this since I switched to...

Provide instructions to use our transport conformance tests with a custom transport

2

See https://github.com/icerpc/icerpc-docs/pull/226#pullrequestreview-1559368409 Perhaps, it's actually more than just instructions. We could consider moving these conformance tests to a separate repository and add a custom transport example.

Investigate flow control based on a dynamic window size

1

I wonder if control flow is the limiting factor here, see https://github.com/dotnet/runtime/issues/53372 and the linked issue https://github.com/dotnet/runtime/issues/43086 Seems similar to what we experience with ice(tcp) vs icerpc(slic) If you capture...

https://github.com/icerpc/icerpc-csharp/blob/89d91b8ec621fcec9125f7c8f7c0d0e67e5e4c54/src/IceRpc/Transports/TcpClientTransport.cs#L44 It would be nice to provide for both client and server transport server addresses more information on which transport parameters are invalid.

See #2820, #2830 and #1411. We need to figure out how an application can easily implement a client/server that provides authentication and authorization for service operations. The proposal to add...

Our generic host example is quite complicated, the server or client needs to load the certificate, setup the ssl client/server authentication options, etc. I think we should consider providing `IHostBuilder`...

Client factory decorators for logging and metrics

2

From a DI perspective, I think it would be much cleaner to replace the `ILogger? logger` client connection or connection cache arguments with an `IClientProtocolConnectionFactory` argument. We'd support a logger...

Improve stream decoding memory allocations

4

In theory, you could decode elements one by one from the payload pipe reader. Right? Afaict, the only reason why we don't do it is to avoid creating a Slice...

Allow the application to provide a callback to check the SSL client hello message?

2

See https://github.com/zeroc-ice/icerpc-csharp/blob/66135594057085d2b2c825fff99f35b957a8a026/src/IceRpc.Quic/Transports/Internal/QuicMultiplexedListener.cs#L89