Project

General

Profile

Actions

Bug #11322

closed

WireGuard Public Key should not be entered by the user

Added by Jim Pingle almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
WireGuard
Target version:
Start date:
01/27/2021
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.5.0
Affected Architecture:

Description

The WireGuard tunnel public key is derived from the private key. There isn't a compelling reason to allow the user to manually enter it separately, as it only presents an additional opportunity for error/misunderstanding.

We should make the field read only and populate it with the value derived from the private key automatically when saving.

The user will need to copy the value, however, so it must still be displayed. We could opt to not store the public key at all, but then that means we would need to run a shell command to get the public key on each page load when editing a WireGuard peer which seems undesirable.

Actions #1

Updated by Jim Pingle almost 4 years ago

  • Status changed from Assigned to In Progress
Actions #2

Updated by Jim Pingle almost 4 years ago

  • Status changed from In Progress to Feedback
  • % Done changed from 0 to 100
Actions #3

Updated by Christian McDonald almost 4 years ago

gitsync'ed and looking good so far

Actions #4

Updated by Marcos M almost 4 years ago

Tested on Jan 29 build. Looks good.

The "Copy" link is a little odd in that it scrolls the viewport when clicking it. I don't recall other places where the functionality is used, so I'm not sure if it's any different.

Actions #5

Updated by Renato Botelho almost 4 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF