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

WebTorrent: Update libdatachannel #5188

Merged

Conversation

paullouisageneau
Copy link
Contributor

This PR is a follow-up on #4123 and updates libdatachannel to fix multiple issues related to connection establishment time (especially when offering to Firefox) and transfer performance.

@paullouisageneau paullouisageneau force-pushed the webtorrent-update-datachannel branch from 48193a8 to 3756c7c Compare September 26, 2020 15:25
@paullouisageneau paullouisageneau force-pushed the webtorrent-update-datachannel branch 4 times, most recently from a00ac5a to 3900dfc Compare September 26, 2020 18:30
@arvidn
Copy link
Owner

arvidn commented Sep 26, 2020

the other PR is applying the peer_list fix and updating the submodule. Since it passed CI, that other PR is sufficient, right? or is this an additional update of the submodule?

@paullouisageneau
Copy link
Contributor Author

@arvidn My mistake, this other PR now applies only the peer_list fix.

@paullouisageneau paullouisageneau force-pushed the webtorrent-update-datachannel branch from 3900dfc to 9eda5e1 Compare September 27, 2020 22:16
@codecov-commenter
Copy link

Codecov Report

Merging #5188 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5188      +/-   ##
==========================================
+ Coverage   65.30%   65.32%   +0.03%     
==========================================
  Files         301      300       -1     
  Lines       32674    32687      +13     
  Branches     7135     7135              
==========================================
+ Hits        21335    21352      +17     
+ Misses       8752     8741      -11     
- Partials     2587     2594       +7     
Impacted Files Coverage Δ
src/ip_voter.cpp 90.24% <0.00%> (-2.44%) ⬇️
src/disk_io_thread_pool.cpp 81.16% <0.00%> (-1.45%) ⬇️
src/identify_client.cpp 80.00% <0.00%> (-1.33%) ⬇️
src/kademlia/routing_table.cpp 82.55% <0.00%> (-0.78%) ⬇️
src/torrent.cpp 55.85% <0.00%> (-0.05%) ⬇️
src/http_connection.cpp 55.52% <0.00%> (-0.04%) ⬇️
src/session_impl.cpp 63.16% <0.00%> (-0.03%) ⬇️
include/libtorrent/optional.hpp
src/mmap_storage.cpp 65.87% <0.00%> (+0.01%) ⬆️
src/http_parser.cpp 92.56% <0.00%> (+0.03%) ⬆️
... and 6 more

@arvidn arvidn merged commit 217128a into arvidn:master Sep 28, 2020
@paullouisageneau paullouisageneau deleted the webtorrent-update-datachannel branch September 28, 2020 07:21
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.

3 participants