Project

General

Profile

Actions

Bug #11407

closed

Removing a WireGuard tunnel in a middle position can break Add button behavior

Added by Jim Pingle about 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
WireGuard
Target version:
-
Start date:
02/12/2021
Due date:
% Done:

100%

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

Description

If there are three WireGuard tunnels (wg0, wg1, wg2) and the middle tunnel (wg1) is removed, the add button links to vpn_wg_edit.php?index=1 since it's trying to incorrectly treat the WireGuard interface number as an array index, which is no longer true. When the user clicks the Add button, it edits the second instance (array index 1) which is now wg2, and does not create a new entry.

Similar to #11323

As a workaround, the user can manually change the URL to remove the index number (e.g. go right to vpn_wg_edit.php with no parameters)

To me, I'm working up a fix.

Actions #1

Updated by Jim Pingle about 3 years ago

  • Description updated (diff)
Actions #2

Updated by Jim Pingle about 3 years ago

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

Updated by Jim Pingle about 3 years ago

  • Target version changed from CE-Next to 2.5.1
Actions #4

Updated by Alhusein Zawi about 3 years ago

  • Status changed from Feedback to Resolved

fixed.

Adding/removing WireGuard tunnels have been done without any issue.

2.5.1-RC (amd64)
built on Thu Mar 11 09:11:02 EST 2021

Actions #5

Updated by Jim Pingle about 3 years ago

  • Target version changed from 2.5.1 to Future
Actions #6

Updated by Jim Pingle about 3 years ago

  • Status changed from Resolved to Waiting on Merge
Actions #7

Updated by Jim Pingle almost 3 years ago

  • Status changed from Waiting on Merge to Closed
Actions #8

Updated by Jim Pingle almost 3 years ago

  • Target version deleted (Future)
Actions

Also available in: Atom PDF