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 tun constants to network defaults #5286

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Conversation

neacsu
Copy link
Contributor

@neacsu neacsu commented Dec 18, 2024

This change is Reviewable

Copy link

vercel bot commented Dec 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-nextra ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 11:38am
nym-next-explorer ⬜️ Ignored (Inspect) Visit Preview Dec 18, 2024 11:38am

@neacsu neacsu requested a review from octol December 18, 2024 11:36
@neacsu neacsu added this to the Hu milestone Dec 18, 2024
@neacsu neacsu force-pushed the feature/mix_tun_const branch from 219f2de to 0532d97 Compare December 18, 2024 11:38
@neacsu neacsu merged commit 3695332 into develop Dec 18, 2024
17 of 18 checks passed
@neacsu neacsu deleted the feature/mix_tun_const branch December 18, 2024 13:03
neacsu added a commit that referenced this pull request Dec 18, 2024
@tommyv1987
Copy link
Contributor

Test Report: Move TUN to Network Constants
Testing Completion Date: 22/01/2025

Testing Steps:

  1. Regression Testing:
    • Verified no issues arose when running tests for the affected files.
    • Tested TUN behaviour with new nym-nodes in the hu branch.

Results:

  • No bugs detected.
  • Tunnels are functioning as expected, with traffic routing and IP generation working seamlessly.

Status: Pass

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