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

Move 'BindingProxies' from 'NetMQTransport' into 'TurnClient' #868

Merged
merged 9 commits into from
May 14, 2020

Conversation

riemannulus
Copy link
Member

This PR should be resolve #756.

@riemannulus riemannulus requested a review from a team May 14, 2020 02:35
Copy link
Contributor

@earlbread earlbread left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add the changelog for this?

Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Copy link
Contributor

@earlbread earlbread left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be good to pass the cancellation token in the missing part.

Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 14, 2020

Codecov Report

Merging #868 into master will increase coverage by 0.01%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #868      +/-   ##
==========================================
+ Coverage   87.53%   87.55%   +0.01%     
==========================================
  Files         250      250              
  Lines       22688    22683       -5     
==========================================
- Hits        19860    19859       -1     
+ Misses       1482     1478       -4     
  Partials     1346     1346              
Impacted Files Coverage Δ
Libplanet.Stun/Stun/NetworkStreamProxy.cs 74.07% <ø> (ø)
Libplanet.Stun/Stun/TurnClient.cs 75.00% <84.61%> (+0.65%) ⬆️
Libplanet/Net/NetMQTransport.cs 83.94% <100.00%> (+0.27%) ⬆️
Libplanet.Tests/Net/Protocols/TestTransport.cs 76.68% <0.00%> (-0.62%) ⬇️
Libplanet.Tests/Net/SwarmTest.cs 95.17% <0.00%> (+0.07%) ⬆️
Libplanet/Net/Protocols/KademliaProtocol.cs 65.01% <0.00%> (+0.38%) ⬆️

@riemannulus riemannulus requested review from dahlia and moreal May 14, 2020 08:05
earlbread
earlbread previously approved these changes May 14, 2020
moreal
moreal previously approved these changes May 14, 2020
longfin
longfin previously approved these changes May 14, 2020
CHANGES.md Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
Libplanet.Stun/Stun/TurnClient.cs Outdated Show resolved Hide resolved
@riemannulus riemannulus dismissed stale reviews from longfin, moreal, and earlbread via a9053fb May 14, 2020 08:40
CHANGES.md Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
Libplanet/Net/NetMQTransport.cs Outdated Show resolved Hide resolved
@earlbread earlbread merged commit c9b9210 into planetarium:master May 14, 2020
@riemannulus riemannulus deleted the separate-turn-stun-feature branch May 14, 2020 09:19
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.

Separate TURN/STUN related features from NetMQTransport
5 participants