Skip to content

Conversation

@Karolk99
Copy link
Contributor

Add mapping function that maps local ip addresses into public ones.
These public addresses are then used to create server reflexive candidates.

@Karolk99 Karolk99 requested review from mickel8 and roznawsk July 23, 2025 12:42
@Karolk99 Karolk99 force-pushed the add-map-to-public-ip-function branch from e87a024 to 8248126 Compare July 23, 2025 12:45
@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.06%. Comparing base (e92cfcf) to head (99fc27a).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
+ Coverage   85.84%   86.06%   +0.22%     
==========================================
  Files          25       25              
  Lines        1872     1902      +30     
==========================================
+ Hits         1607     1637      +30     
  Misses        265      265              
Files with missing lines Coverage Δ
lib/ex_ice/ice_agent.ex 65.75% <ø> (ø)
lib/ex_ice/priv/gatherer.ex 95.71% <100.00%> (+2.38%) ⬆️
lib/ex_ice/priv/ice_agent.ex 87.73% <100.00%> (+0.05%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e92cfcf...99fc27a. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Karolk99 Karolk99 requested review from mickel8 July 23, 2025 16:15
@ipv6 {64_512, 0, 0, 0, 0, 0, 0, 1}
@invalid_ip :invalid_ip

test "adds srflx candidate" do
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, all tests except for the first one and the last one could be moved to gatherer_test.exs

Copy link
Member

@roznawsk roznawsk left a comment

Choose a reason for hiding this comment

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

LGTM.

@Karolk99 Karolk99 merged commit a4f930b into master Jul 24, 2025
4 checks passed
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.

4 participants