Project

General

Profile

Bug #11312

Unable to edit or add WireGuard peers

Added by RED SKULL about 1 month ago. Updated about 1 month ago.

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

100%

Estimated time:
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

Associated revisions

Revision 7e226dc7 (diff)
Added by Jim Pingle about 1 month ago

Encode WireGuard tunnel list values. Issue #11312

Revision cd4103cd (diff)
Added by Jim Pingle about 1 month ago

Encode WireGuard tunnel edit/peer values. Issue #11312

History

#1 Updated by RED SKULL about 1 month 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.

#2 Updated by Jim Pingle about 1 month 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.

#3 Updated by Jim Pingle about 1 month 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.

#4 Updated by RED SKULL about 1 month 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.

#5 Updated by Marcos Mendoza about 1 month ago

Tested on Jan 29 build. Looks good.

#6 Updated by Marcos Mendoza about 1 month ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF