Project

General

Profile

Actions

Regression #12398

closed

"Expiration and Replacement" section is shown twice when editing a mobile IPsec phase 2 entry

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

Status:
Resolved
Priority:
High
Assignee:
Category:
IPsec
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
22.01
Release Notes:
Force Exclusion
Affected Version:
Affected Architecture:

Description

When editing a phase 2 entry for a mobile IPsec tunnel the "Expiration and Replacement" section is shown twice.

This is due to the "Keep Alive" section being added overlapping inside the section for hidden items, it should be moved outside.

This may also break mobile P2s as it ends up losing the internal mobile type flag on the P2.

Actions #1

Updated by Jim Pingle over 3 years ago

  • Target version changed from 2.6.0 to CE-Next
  • Plus Target Version changed from 21.09 to 22.01

The type bit wasn't the case, it was the remoteid type which was lost but that isn't necessary from what I can tell. The other hidden values are added as globals not a part of a section, so they are fine.

This is only cosmetic from the looks of it, so no need to squeeze it in.

Actions #2

Updated by Jim Pingle over 3 years ago

  • Status changed from New to Feedback

Fixed in 863ab7d4

Actions #3

Updated by Danilo Zrenjanin about 3 years ago

I tested on the 2.5.2-RELEASE (amd64). With all defaults settings under Phase 2, I couldn't replicate it.

Is there any particular setup that provokes it to show up twice?

Actions #4

Updated by Jim Pingle about 3 years ago

  • Tracker changed from Bug to Regression

It won't show on 2.5.2, only on recent builds of 2.6.0 (after I introduced the bug and before I fixed it) and on current builds of 21.09.

Actions #5

Updated by Max Leighton about 3 years ago

  • Status changed from Feedback to Resolved

I checked and saw it in 21.09. The fix works. The Expiration and Replacement section only appears once in 22.01:

22.01-DEVELOPMENT (amd64)
built on Tue Sep 28 01:14:40 EDT 2021
FreeBSD 12.2-STABLE

Marking the ticket resolved.

Actions #6

Updated by Jim Pingle about 3 years ago

  • Target version changed from CE-Next to 2.6.0
  • Release Notes changed from Default to Force Exclusion
Actions

Also available in: Atom PDF