Skip to content

wireguard creds fix #2439

Merged
Marketen merged 1 commit into
developfrom
marc/wireguard-creds-fix
Jun 10, 2026
Merged

wireguard creds fix #2439
Marketen merged 1 commit into
developfrom
marc/wireguard-creds-fix

Conversation

@Marketen

Copy link
Copy Markdown
Contributor

This pull request refactors how WireGuard device configuration is fetched and displayed, introducing a new API route to retrieve either the remote or local configuration for a device. The UI is updated to handle loading and error states for local configs, and the backend logic is simplified and made more robust. The main changes are grouped below:

API and Backend Enhancements:

  • Added a new API route wireguardDeviceConfigGet to fetch either the remote or local configuration for a WireGuard device, and updated the Routes interface and route data accordingly. [1] [2] [3] [4] [5] [6]
  • Refactored backend logic to separate fetching of remote and local configs, added device existence checks, improved error handling with logging, and trimmed device names to avoid empty entries. [1] [2] [3]

Frontend/UI Improvements:

  • Updated the device details UI to use the new wireguardDeviceConfigGet route, allowing independent fetching of local and remote configs, and added UI feedback for loading and error states when retrieving local credentials. [1] [2] [3] [4]
  • Refactored the download handler to use the new config API and improved the logic for determining if the local config should be fetched.

These changes make the WireGuard configuration retrieval more modular, robust, and user-friendly.

@Marketen Marketen requested a review from a team as a code owner June 10, 2026 15:02
@Marketen Marketen merged commit 72f4a01 into develop Jun 10, 2026
6 checks passed
@Marketen Marketen deleted the marc/wireguard-creds-fix branch June 10, 2026 15:03
@github-actions

Copy link
Copy Markdown

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.

1 participant