fix: use stored site data for WireGuard credentials display #151

Closed
opened 2026-04-05 17:01:38 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @LaurenceJJones on 3/5/2026

Fixes #2341

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

When viewing an existing WireGuard site's credentials, the page was
fetching pick-site-defaults which returns data for a random exit node,
causing the displayed interface IP address to differ from what was
shown during site creation.

Now uses the site's stored data directly from the database for the
obfuscated config display. Regeneration still fetches new defaults
as expected.

How to test?

When viewing the generated credentials instead of requesting new data each time and the or condition taking over it should display the stored credentials.

*Originally created by @LaurenceJJones on 3/5/2026* Fixes #2341 ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description When viewing an existing WireGuard site's credentials, the page was fetching pick-site-defaults which returns data for a random exit node, causing the displayed interface IP address to differ from what was shown during site creation. Now uses the site's stored data directly from the database for the obfuscated config display. Regeneration still fetches new defaults as expected. ## How to test? When viewing the generated credentials instead of requesting new data each time and the `or` condition taking over it should display the stored credentials.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/pangolin#151