-
Notifications
You must be signed in to change notification settings - Fork 582
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
refactor: Improve handshake resiliency of peer #95
Conversation
Codecov Report
@@ Coverage Diff @@
## main #95 +/- ##
==========================================
+ Coverage 71.65% 72.15% +0.50%
==========================================
Files 89 89
Lines 3612 3627 +15
Branches 55 55
==========================================
+ Hits 2588 2617 +29
+ Misses 799 791 -8
+ Partials 225 219 -6
Continue to review full report at Codecov.
|
Going to be bad and use admin privileges to merge this. The peer flake is actively hurting other PRs, so seems reasonable. |
9758be4
to
1c61fef
Compare
We were depending on the close mutex to properly report connection state. This ensures the RTC connection is properly closed before returning.
00f49f3
to
3367212
Compare
006ab67
to
8b33e86
Compare
b493807
to
e5cb06b
Compare
c5c8b65
to
b69f269
Compare
7df4cc9
to
e87d2cb
Compare
93ce57e
to
9ef1d89
Compare
We were depending on the close mutex to properly
report connection state. This ensures the RTC
connection is properly closed before returning.