Project

General

Profile

Actions

Bug #11312

closed

Unable to edit or add WireGuard peers

Added by RED SKULL about 3 years ago. Updated about 3 years ago.

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

100%

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

Description

After upgrading from test build 2.5.0.a.20210122.2350 to 2.5.0.a.20210125.0856:

-- I am unable to edit all existing peers under the sole existing tunnel I created under the prior build. After clicking the edit button, nothing happens (window containing peer variables is not shown)
-- "Edit Peers" button does not have any effect
-- "Save" does not have any effect

Actions #1

Updated by RED SKULL about 3 years ago

Ability to edit wireguard peers was regained by editing /cf/conf/config.xml and removing values from the following variables

-- <descr></descr>

-- <persistentkeepalive></persistentkeepalive>

It seems as though certain strings/characters I entered for those variables caused the issue.

Actions #2

Updated by Jim Pingle about 3 years ago

  • Subject changed from Unable to edit or add peers - Build 2.5.0.a.20210125.0856 (01/25/21) to Unable to edit or add WireGuard peers
  • Assignee set to Jim Pingle

What did you have in those fields?

The keepalive value is numeric so it's unlikely to be that. Description is CDATA encoded so also unlikely to be a source of problems, but perhaps depending on the contents it may have triggered something. Without knowing what that was, it's difficult to say.

Actions #3

Updated by Jim Pingle about 3 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

I found a couple issues on the page that could be a problem if the description contained a single quote (') which could explain the behavior you observed. I pushed some fixes, which should make it into snapshots later today/overnight.

Actions #4

Updated by RED SKULL about 3 years ago

I had an string that included a single quote encased by the <descr></descr> variable so it lines up perfectly with your observation. Thank you.

Actions #5

Updated by Marcos M about 3 years ago

Tested on Jan 29 build. Looks good.

Actions #6

Updated by Marcos M about 3 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF