Skip to content

feat(ipsec): convert DPD to combobox#768

Merged
Tbaile merged 1 commit into
nethsecurity-8.8from
feat/ipsec-dpd-combobox
May 15, 2026
Merged

feat(ipsec): convert DPD to combobox#768
Tbaile merged 1 commit into
nethsecurity-8.8from
feat/ipsec-dpd-combobox

Conversation

@gsanchietti
Copy link
Copy Markdown
Member

@gsanchietti gsanchietti commented May 15, 2026

Convert DPD radio to NeCombobox with options clear/trap/restart.

Fixes: NethServer/nethsecurity#1298
Related PR (no required for the UI): NethServer/nethsecurity#1676

Screenshot:
image

Available options:
image

@gsanchietti gsanchietti force-pushed the feat/ipsec-dpd-combobox branch from 09a24f8 to fafd112 Compare May 15, 2026 10:33
@gsanchietti gsanchietti requested a review from Tbaile May 15, 2026 10:40
@gsanchietti gsanchietti changed the base branch from main to nethsecurity-8.8 May 15, 2026 10:40
@gsanchietti gsanchietti force-pushed the feat/ipsec-dpd-combobox branch from fafd112 to fa8abb1 Compare May 15, 2026 12:14
Copy link
Copy Markdown
Collaborator

@Tbaile Tbaile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the need for a lib for a single method that is being used only on the tunnel drawer?

Changes:
- expose all possibile values for dpd action
- add tooltip to explain the option values

Assysted-by: Copilot:GPT5.4
@gsanchietti gsanchietti force-pushed the feat/ipsec-dpd-combobox branch from fa8abb1 to 65980b9 Compare May 15, 2026 12:22
@gsanchietti gsanchietti requested a review from Tbaile May 15, 2026 12:22
@Tbaile Tbaile merged commit d95a3d9 into nethsecurity-8.8 May 15, 2026
5 checks passed
@Tbaile Tbaile deleted the feat/ipsec-dpd-combobox branch May 15, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid configuration: dpd_action = start accepted but not supported by strongSwan

2 participants