Skip to content

Lp/ip pool fixes#6412

Merged
jstuczyn merged 3 commits intorelease/2026.3-parmigianofrom
lp/ip-pool-fixes
Feb 5, 2026
Merged

Lp/ip pool fixes#6412
jstuczyn merged 3 commits intorelease/2026.3-parmigianofrom
lp/ip-pool-fixes

Conversation

@jstuczyn
Copy link
Contributor

@jstuczyn jstuczyn commented Feb 3, 2026

this PR fixes the IpPool and more importantly unifies the process of Peer registration between the Authenticator and LP through the now shared LpRegistrator.

In general the registration has the following structure:

  1. Initial request message is received
    1.1. We check if the peer has already registered before -> if so, we returned the past information
    1.2. We check if the peer already has a pending registration -> if so, we return the past information
    1.3. We pre-allocated [nym_wireguard::ip_pool::IpPair] and save time-sensitive pending registration. If it does not complete within specified time interval, the information is going to get removed.
  2. Finalisation request message is received, where credential has to be attached is verified. Upon successful completion, pending registration is transformed into a properly inserted peer.

This change is Reviewable

@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nym-explorer-v2 Ready Ready Preview, Comment Feb 5, 2026 2:34pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs-nextra Ignored Ignored Preview Feb 5, 2026 2:34pm
nym-node-status Ignored Ignored Preview Feb 5, 2026 2:34pm

Request Review

removed unused imports

gateway probe fixes

PSK injection + test fixes

cleanup minus PSK injection

combine with lp reg

moved authenticator peer registration to centralised location

bugfix: ensure IpPool never allocates gateway ip

ip pool allocation tests
@jstuczyn jstuczyn merged commit a151a03 into release/2026.3-parmigiano Feb 5, 2026
19 checks passed
@jstuczyn jstuczyn deleted the lp/ip-pool-fixes branch February 5, 2026 14:47
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.

2 participants