Skip to content

feat: Add groupchat API function that returns an IP address string for a peer#2446

Merged
toktok-releaser merged 1 commit into
TokTok:masterfrom
JFreegman:group_ip_addr
Dec 19, 2023
Merged

feat: Add groupchat API function that returns an IP address string for a peer#2446
toktok-releaser merged 1 commit into
TokTok:masterfrom
JFreegman:group_ip_addr

Conversation

@JFreegman
Copy link
Copy Markdown
Member

@JFreegman JFreegman commented Dec 1, 2023

This function will return an IP address string associated with a peer.
If the peer is not accepting direct connections a placeholder value
will be returned, indicating that their real IP address is unknown.
We do not return TCP relay IP addresses because a TCP connection
with a peer may use multiple relays simultaneously.


This change is Reviewable

@JFreegman JFreegman added the enhancement New feature for the user, not a new feature for build script label Dec 1, 2023
@JFreegman JFreegman added this to the v0.2.19 milestone Dec 1, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 1, 2023

Codecov Report

Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (5f863a5) 70.02% compared to head (b2315c5) 70.09%.

Files Patch % Lines
toxcore/tox_private.c 66.66% 10 Missing ⚠️
toxcore/group_chats.c 78.57% 6 Missing ⚠️
toxcore/tox_api.c 0.00% 2 Missing ⚠️
toxcore/network.c 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2446      +/-   ##
==========================================
+ Coverage   70.02%   70.09%   +0.07%     
==========================================
  Files          75       75              
  Lines       25825    25890      +65     
==========================================
+ Hits        18083    18147      +64     
- Misses       7742     7743       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JFreegman JFreegman force-pushed the group_ip_addr branch 23 times, most recently from 00d5c40 to 9846f38 Compare December 5, 2023 18:59
@Green-Sky
Copy link
Copy Markdown
Member

relates to #2422

@JFreegman JFreegman force-pushed the group_ip_addr branch 3 times, most recently from 1f259de to 7b943a2 Compare December 13, 2023 17:03
@JFreegman JFreegman force-pushed the group_ip_addr branch 4 times, most recently from ae024f4 to 5964df6 Compare December 18, 2023 20:42
@JFreegman JFreegman force-pushed the group_ip_addr branch 2 times, most recently from 420004e to a67d03f Compare December 19, 2023 16:05
This function will return an IP address string associated with a peer.
If the peer is not accepting direct connections a placeholder value
will be returned, indicating that their real IP address is unknown.
We do not return TCP relay IP addresses because a TCP connection
with a peer may use multiple relays simultaneously.
@toktok-releaser toktok-releaser merged commit b2315c5 into TokTok:master Dec 19, 2023
@JFreegman JFreegman deleted the group_ip_addr branch December 19, 2023 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature for the user, not a new feature for build script

Development

Successfully merging this pull request may close these issues.

4 participants