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: bug causing friend group invites to sometimes fail & improve logging #2338

Merged
merged 1 commit into from
Oct 1, 2022

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Oct 1, 2022

The group privacy status was incorrectly set to private when a peer accepted a friend's group invite, which would cause handshake requests to fail in certain scenarios


This change is Reviewable

@JFreegman JFreegman added the bug Bug fix for the user, not a fix to a build script label Oct 1, 2022
@JFreegman JFreegman added this to the v0.2.19 milestone Oct 1, 2022
@codecov
Copy link

codecov bot commented Oct 1, 2022

Codecov Report

Merging #2338 (856e4fe) into master (8054854) will increase coverage by 0.02%.
The diff coverage is 41.66%.

@@            Coverage Diff             @@
##           master    #2338      +/-   ##
==========================================
+ Coverage   78.12%   78.15%   +0.02%     
==========================================
  Files         140      140              
  Lines       31045    31053       +8     
==========================================
+ Hits        24255    24268      +13     
+ Misses       6790     6785       -5     
Impacted Files Coverage Δ
toxcore/group_chats.c 69.18% <41.66%> (-0.10%) ⬇️
toxcore/TCP_client.c 83.37% <0.00%> (-0.71%) ⬇️
toxcore/onion_client.c 89.76% <0.00%> (-0.12%) ⬇️
toxcore/DHT.c 85.08% <0.00%> (-0.08%) ⬇️
toxcore/group.c 84.48% <0.00%> (+0.06%) ⬆️
toxcore/net_crypto.c 87.36% <0.00%> (+0.07%) ⬆️
toxav/toxav.c 69.23% <0.00%> (+0.14%) ⬆️
toxcore/friend_connection.c 85.64% <0.00%> (+0.23%) ⬆️
auto_tests/group_sync_test.c 85.11% <0.00%> (+0.46%) ⬆️
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

The group privacy status was incorrectly set to private when a peer
accepted a friend's group invite, which would cause handshake requests
to fail in certain scenarios
@JFreegman JFreegman merged commit 856e4fe into TokTok:master Oct 1, 2022
@JFreegman JFreegman deleted the ngc_invite_bug_fix branch December 7, 2023 22:57
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fix for the user, not a fix to a build script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants