attempt to stabilize CopyToAsync_AllDataCopied_Large for Tls13

[wfurt]

I'm not 100% what is going on but with this change I did several hundreds runs and I did not see any failure.
The TLS1.3 are not quite symmetric as 1.2.

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Details

Issue Details

Description:	I'm not 100% what is going on but with this change I did several hundreds runs and I did not see any failure. The TLS1.3 are not quite symmetric as 1.2.
Author:	wfurt
Assignees:	-
Labels:	area-System.Net.Security
Milestone:	-

[wfurt]

/azp run runtime-libraries-coreclr outerloop-linux

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[geoffkizer]

Why does this make it better? Seems like it shouldn't matter.

[wfurt]

I'm still digging into it. But here are some (unverified) thoughts. For TLS 1.2 (and less) when handshake is done the streams are really identical and fully interchangeable. With TLS1.3 part of the handshake piggy-backs on first data chunks - that was done to limit round trips and that is reason why we always hit the "RENEGOTIATION" path on Windows. The test sends unidirectional data and and I'm not sure if we have mechanism to send message unless somebody call encrypt data.

Now the interesting part:
The session fails because the writer sends RST. Since the NetworkStream owns the socket, it should call shutdown on dispose and I would expect that would still do normal TCP close using FIN.

[geoffkizer]

Yeah, very weird about the RST.

[wfurt]

I think it may be timing after all @geoffkizer . I opened #44764 to track what I see as underlying problem.

[wfurt]

full details are inside #44764. This change should be sufficient to make the test reliable. Alternatively, we would need to change the test to read extra TLS frame(s) from the buffer before Disposing writer.

fixes #44175

[wfurt]

/azp run runtime-libraries-coreclr outerloop-linux

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[wfurt]

/azp run runtime-libraries-coreclr outerloop-linux

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[wfurt]

/azp run runtime-libraries-coreclr outerloop-linux

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[wfurt]

test failures are unrelated.