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 errors on TurnClient #972

Merged
merged 3 commits into from
Aug 28, 2020
Merged

Fix errors on TurnClient #972

merged 3 commits into from
Aug 28, 2020

Conversation

longfin
Copy link
Member

@longfin longfin commented Aug 27, 2020

This PR addresses #957 and #969.

@longfin longfin added bug Something isn't working network Related to networking (Libplanet.Net) labels Aug 27, 2020
@longfin longfin self-assigned this Aug 27, 2020
@longfin longfin marked this pull request as ready for review August 27, 2020 12:36
@longfin
Copy link
Member Author

longfin commented Aug 27, 2020

fe0f6f5 misaddresses #969 that covered by #906. (TurnClient can't be recoverable itself 😓)

but I guess some tweak-able points remains...

  • Reconnect to TURN when the connection is lost #909 covered NetMQTransport.CreatePermission() well, but other cases that using NetMQTransport._turnClient aren't.
  • Reconnecting doesn't wait all tasks related to TURN relay. (however, I can't wait them too because they didn't finish 😢 )

so I'll remove this commit about #969 from this PR and create another PR for that.

@codecov
Copy link

codecov bot commented Aug 27, 2020

Codecov Report

Merging #972 into main will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #972      +/-   ##
==========================================
- Coverage   88.73%   88.71%   -0.03%     
==========================================
  Files         315      315              
  Lines       27683    27687       +4     
==========================================
- Hits        24565    24563       -2     
- Misses       1573     1577       +4     
- Partials     1545     1547       +2     
Impacted Files Coverage Δ
Libplanet.Stun/Stun/TurnClient.cs 76.95% <100.00%> (ø)
Libplanet.Tests/Net/SwarmTest.cs 95.41% <100.00%> (+0.01%) ⬆️
Libplanet/Net/Protocols/KademliaProtocol.cs 69.53% <0.00%> (-0.41%) ⬇️
Libplanet.RocksDBStore/RocksDBStore.cs 95.56% <0.00%> (-0.32%) ⬇️
Libplanet.Tests/Net/SwarmTest.Preload.cs 99.01% <0.00%> (-0.18%) ⬇️

earlbread
earlbread previously approved these changes Aug 28, 2020
moreal
moreal previously approved these changes Aug 28, 2020
@longfin
Copy link
Member Author

longfin commented Aug 28, 2020

/rebase

riemannulus
riemannulus previously approved these changes Aug 28, 2020
earlbread
earlbread previously approved these changes Aug 28, 2020
moreal
moreal previously approved these changes Aug 28, 2020
limebell
limebell previously approved these changes Aug 28, 2020
@earlbread earlbread merged commit 6c4ecb0 into planetarium:main Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working network Related to networking (Libplanet.Net)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants