OrleansContrib / SignalR.Orleans

SignalR backend based on Orleans.
MIT License
295 stars 64 forks source link

Remove ArrayPool in ClientGrain.OnActivateAsync and replace with new allocated array #134

Closed michaelmcneilnet closed 2 years ago

michaelmcneilnet commented 2 years ago

ArrayPool returns an array which is at least the requested length, and so can have null elements at the end. Task.WhenAll throws an exception when provided an array with nulls.

Replaces #133 which is fixing the same issue but by keeping the arraypool and filtering out the nulls.

This solves #128