feat/ntm: NIP-13 Exit policy upgrade - #6959
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
📝 WalkthroughWalkthroughThe WireGuard exit-policy allowlist in ChangesWireGuard exit port allowlist
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@scripts/nym-node-setup/network-tunnel-manager.sh`:
- Around line 929-937: Reactivate PORT_MAPPINGS entry RainbowSixSiege1 at
scripts/nym-node-setup/network-tunnel-manager.sh lines 929-937. Also reactivate
MQTTS at lines 952-954, and keep DiscordVoiceChat1’s uncovered 65001-65535
portion active at lines 967-1004; preserve the existing governance grouping
comments and mapping structure.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: c546058a-c4ff-42ca-99b1-9c9d86f78d8c
📒 Files selected for processing (1)
scripts/nym-node-setup/network-tunnel-manager.sh
This PR adds ports from NIP-13 to Exit policy managed by operators through NTM
This change is
Summary by CodeRabbit