Closed dougbu closed 1 year ago
This version looks fine, but you say you're still seeing other issues?
Not yet. I suspect there be a few more gremlins in HttpMessageContent
, at least wrt. HttpMessageContent.ValidateStreamForReading(...)
. I haven't filed an issue about that or anything else my testing may turn up today. My plan is to do so and fix what I find in another PR.
net6.0
tests disabled in #384 for this issueStream.CanSeek
less when calculatingHttpMessageContent.ContentLength
valueHttpContent.Headers.ContentLength
if availableHttpMessageContent.Headers.ContentLength!=null
for empty messagesContentLength==null
versusContentLength==0
inconsistencies (depending on platform)ContentLength==null
may still occur in some corner cases (which existed before)HttpContent
doesn't know its length andReadAsStreamAsync()
hasn't completedHttpResponseMessage
s we expose to user code are consistent across platformsEmptyContent
inHttpResponseMessage
s we create