ffang commented on PR #3554: URL: https://github.com/apache/cxf/pull/3554#issuecomment-6060848718
Thanks @vp340 for digging into this. I agree with @reta here: closing all streams in postClose() and keeping the close failure as a suppressed exception is the right behaviour, since that suppressed exception carries accurate information. Skipping the flow-through close leaves the transport stream open, and only the second close() would close it, which may never happen once the fault chain has switched streams. With #3555, writes after close fail straight away, so I think that covers the concern without the extra state. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
