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

Fix Reconnect Crash By Removing Waiting For WebSocket Connection #69

Closed
wants to merge 8 commits into from

Conversation

lastpeony
Copy link
Contributor

@lastpeony lastpeony commented Apr 17, 2024

Call init by default on webrtcClieint initialization

Call init inside publish/play only if sdk is closed previously.(stop called)

Remove waiting for WS connection with thread.sleep, move to callback style

ant-media/Ant-Media-Server#6100

reconnection rewritten
segmentation fault crash fixed
waiting for ws connection removed. ANR crash due to it fixed.
by default reconnection is enabled
users can call publish/play consequently many times, even if ws is not connected.

@lastpeony lastpeony requested a review from burak-58 April 17, 2024 14:32
@lastpeony
Copy link
Contributor Author

do not review yet this actually didnt solve the bug.
the root cause is harder than i thought.

@lastpeony
Copy link
Contributor Author

lastpeony commented Apr 22, 2024

more changes added. segmentation fault fixed. i performed some comprehensive testing on my side and it worked fine. its reviewable now. @burak-58

@lastpeony lastpeony requested review from burak-58 and removed request for burak-58 May 21, 2024 12:52
@lastpeony lastpeony force-pushed the reconnectCrashFix branch 2 times, most recently from 816bb73 to 33b8128 Compare June 6, 2024 19:45
@lastpeony
Copy link
Contributor Author

This PR is refactored and broken down to smaller PRs.

@lastpeony lastpeony closed this Jul 1, 2024
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.

1 participant