Skip to content

Conversation

@aptalca
Copy link
Member

@aptalca aptalca commented Jan 24, 2026

testing locally

Tested locally with 2 wireguard containers on 2 different VPSes:

  • Set first one up with PEER=1, PEERDNS=auto, INTERNAL_SUBNET=10.13.18.0 and URL=mydomain.duckdns.org
  • Set the second one up with no env vars and the conf from first server dropped in (contains DNS = 10.13.18.1)
  • Started the second container, it connects to the first via duckdns domain:
    • wg show on the first shows the handshake
    • wg show on the second shows the first one's IP in endpoint (correct url resolution)
    • curl icanhzip.com on the second returns the first one's IP
    • resolv.conf contains the header # Generated by resolvconf and nameserver 10.13.18.1
    • Exec'ed in and did wg_quick down wg1, resolv.conf contents reverted to docker default with my host's search domains
  • Then removed the conf from the second and restarted with an invalid conf, resolv.conf was docker's original with my search domains, but with the header added # Generated by resolvconf

PS.
I also noticed that the init script fails with a confusing message if no wg conf is supplied:

**** Found WG conf /config/wg_confs/*.conf, but it doesn't seem to be valid, skipping. ****
**** No valid tunnel config found. Please create a valid config and restart the container ****

We should probably check for that condition and log a message, but that can be in a different PR.

@LinuxServer-CI
Copy link
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-wireguard:1.0.20250521-r1-pkg-1e8578fc-dev-0b19f44326665d275ca466f4e454d95331ce3649-pr-402

@LinuxServer-CI
Copy link
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-wireguard:1.0.20250521-r1-pkg-1e8578fc-dev-ad124609248ce6aff978110e28d5ac9b97049be4-pr-402

@aptalca aptalca marked this pull request as ready for review January 24, 2026 01:57
@aptalca aptalca requested a review from a team January 24, 2026 01:57
@LinuxServer-CI LinuxServer-CI moved this from PRs to PRs Ready For Team Review in Issue & PR Tracker Jan 24, 2026
@github-project-automation github-project-automation bot moved this from PRs Ready For Team Review to PRs Approved in Issue & PR Tracker Jan 24, 2026
@aptalca aptalca merged commit cd70ced into master Jan 31, 2026
6 checks passed
@aptalca aptalca deleted the alpine3.23 branch January 31, 2026 23:11
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Jan 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants