Skip to content
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

fixed memory leak and added TSL workaround for iOS 5.0 #98

Closed
wants to merge 7 commits into from

Conversation

adrianUken
Copy link

-renamed the block variable "self" to "webSocket" in several blocks to prevent confusion
-ensured references in blocks to self (and properties of self) properly reference the "webSocket" block variable to prevent retain loops
-added the TSL workaround for iOS 5.0 devices to properly connect on secure connections as described here https://developer.apple.com/library/ios/#technotes/tn2287/_index.html#//apple_ref/doc/uid/DTS40011309

fixed memory leak if socket fails connection in secure mode, added worka...
@JakeWharton
Copy link
Contributor

This needs a rebase and there's a bunch of commits which shouldn't have been added. Please re-submit a pull request if this is still an issue.

@shyreen
Copy link

shyreen commented Nov 19, 2015

This solution is not working for wss on IOS 9.1, i'm getting CFNetwork SSLHandshake failed (-9847) error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants