github-starred/pangolin
2
0
Fork 0
mirror of https://github.com/fosrl/pangolin.git synced 2026-05-07 13:19:07 -05:00
Code Issues 564 Packages Projects Releases 72 Wiki Activity

[PR #2643] [MERGED] fix: Display actual values for WireGuard site credentials #5967

New Issue
Closed
opened 2026-04-23 02:54:36 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-23 02:54:36 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/2643
Author: @Fredkiss3
Created: 3/12/2026
Status: ✅ Merged
Merged: 3/29/2026
Merged by: @miloschwartz

Base: dev ← Head: fix/wireguard-site-ip

📝 Commits (1)

  • 84b082e ♻️ show actual values for wireguard site credentials whenever possible

📊 Changes

2 files changed (+11 additions, -15 deletions)

View changed files

📝 src/app/[orgId]/settings/sites/[niceId]/credentials/page.tsx (+10 -12)
📝 src/lib/wireguard.ts (+1 -3)

📄 Description

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.

Summary

Changes:

  • Show the actual values from the DB in wireguard site credentials
  • Also made it so that the public key in wireguard site credentials is obfuscated
  • Bug fix: When regenerating credentials for sites using wireguard, the public key shown always abc123 instead the actual generated public key

fixes #2341

Screenshots

Name Screenshots
Obfuscated wireguard credentials Screenshot 2026-03-12 at 23 40 09
Obfuscated wireguard credentials (in DB) Screenshot 2026-03-12 at 23 40 29
Regenerated public key before Screenshot 2026-03-12 at 23 46 30
Regenerated public key after Screenshot 2026-03-12 at 23 46 51

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/fosrl/pangolin/pull/2643 **Author:** [@Fredkiss3](https://github.com/Fredkiss3) **Created:** 3/12/2026 **Status:** ✅ Merged **Merged:** 3/29/2026 **Merged by:** [@miloschwartz](https://github.com/miloschwartz) **Base:** `dev` ← **Head:** `fix/wireguard-site-ip` --- ### 📝 Commits (1) - [`84b082e`](https://github.com/fosrl/pangolin/commit/84b082e1941b243f4aca9e193abc81a083c013a9) ♻️ show actual values for wireguard site credentials whenever possible ### 📊 Changes **2 files changed** (+11 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `src/app/[orgId]/settings/sites/[niceId]/credentials/page.tsx` (+10 -12) 📝 `src/lib/wireguard.ts` (+1 -3) </details> ### 📄 Description ## 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. ## Summary **Changes**: - Show the actual values from the DB in wireguard site credentials - Also made it so that the public key in wireguard site credentials is obfuscated - Bug fix: When regenerating credentials for sites using wireguard, the public key shown always `abc123` instead the actual generated public key fixes #2341 ## Screenshots | Name | Screenshots | | :----------------: | :-----------: | | Obfuscated wireguard credentials | <img width="1475" height="1061" alt="Screenshot 2026-03-12 at 23 40 09" src="https://github.com/user-attachments/assets/96e78e1c-205b-40b4-b66b-e30e56e2e1f8" /> | | Obfuscated wireguard credentials (in DB) | <img width="1475" height="1061" alt="Screenshot 2026-03-12 at 23 40 29" src="https://github.com/user-attachments/assets/d468a595-817c-4c54-bf13-93692325f3f5" /> | | Regenerated public key before | <img width="1028" height="524" alt="Screenshot 2026-03-12 at 23 46 30" src="https://github.com/user-attachments/assets/cdbe8691-44de-4124-89a7-b23b2cfeec3a" /> | | Regenerated public key after | <img width="1035" height="575" alt="Screenshot 2026-03-12 at 23 46 51" src="https://github.com/user-attachments/assets/6f351de7-9085-4984-9e10-027791dd9d57" /> | --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-23 02:54:36 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
api
authentication
bug
config
dependencies
docker
documentation
enhancement
good first issue
help wanted
Improvement
Look Into
needs investigating
networking
new feature
non-critical bug
potential bug
pull-request
Mirrored from GitHub Pull Request
 question
reverse proxy
Security
stale
ui
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#5967
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?