You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think to make the library closer to a drop in replacement to MemoryStream it would be good if double dispose wasn't logged as a critical, maybe information instead.
I understand the rationale for logging it and agree that it's useful, but given that double dispose is explicitly called out as being fine in the Stream docs, Asp.Net Core double disposes returned streams, and there's no negative impact in practice, I think critical seems like an overly severe categorisation.