-
Notifications
You must be signed in to change notification settings - Fork 1.2k
URLSessionWebSocketTask.receive() not finishing if server closes connection without a close packet #4673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ection without a close packet
@jrflat could you please review? |
@swift-ci please test |
@@ -7,8 +7,13 @@ | |||
// See http://swift.org/CONTRIBUTORS.txt for the list of Swift project authors | |||
// | |||
|
|||
import Foundation | |||
import XCTest | |||
#if NS_FOUNDATION_ALLOWS_TESTABLE_IMPORT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was necessary for TestDecimal to build on macOS against the correct Foundation library. See also e9e183c#r1011034654 . I was mistaken in thinking this change was unnecessary.
The macOS build failure appears unrelated to this change?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@swift-ci please test |
Same macOS error :-/
|
@swift-ci please test macOS platform |
Seems the macOS platform build is still having issues unrelated to this change? |
@swift-ci please test macOS platform |
|
@swift-ci please test macOS platform |
Tests look good! |
Two fixes required here:
doPendingWork()
if the connection is closed out due to an error, in order to respond to any pending Tasks parked inreceive()