Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OpenVPN tunnel client : wrong "Bridged" mode as new default #752

Closed
cotosso opened this issue Sep 11, 2024 · 2 comments
Closed

OpenVPN tunnel client : wrong "Bridged" mode as new default #752

cotosso opened this issue Sep 11, 2024 · 2 comments
Assignees
Labels
verified All test cases were verified successfully

Comments

@cotosso
Copy link
Contributor

cotosso commented Sep 11, 2024

One of last fix changed the OpenVPN tunnel client default from Routed to Bridged.
To avoid problems is better to revert to previous behaviour cause openvpn tunnel client are almost always used in Routed mode and with this new behaviour admins can easily make mistakes.

Another side question is: cause Routed mode is by far the most used it should be presented as first option in the radio button interface (for both OpenVPN tunnel client and OpenVPN roadwarrior panel).

Steps to reproduce

  • Create an openvpn tunnel client from scratch
  • don't even open the "advanced settings"
  • press save button and confirm

Expected behavior

A new openvpn tunnel client in Routed mode has been created.

Actual behavior

A new openvpn tunnel client in Bridged Mode has been created,

Components

NethSecurity 8 23.05.4-ns.1.2.0-36-gfcc1f065

@cotosso cotosso added the bug label Sep 11, 2024
@cotosso cotosso changed the title OpenVPN tunnel client : wrong "Bridged" as new default OpenVPN tunnel client : wrong "Bridged" mode as new default Sep 11, 2024
@Tbaile
Copy link
Contributor

Tbaile commented Sep 12, 2024

Without releasing a new ns-ui package yet (since this issue has been deemed not very critical), please install this ui in a stable firewall to check if the fixes are applied: https://github.com/NethServer/nethsecurity-ui/actions/runs/10825906186/artifacts/1923364440

\cc @cotosso

@Tbaile Tbaile added the testing Packages are available from testing repositories label Sep 12, 2024
@gsanchietti gsanchietti moved this to In progress 🛠 in NethSecurity Sep 12, 2024
@gsanchietti gsanchietti added this to the NethSecurity 8.3 milestone Sep 12, 2024
@gsanchietti
Copy link
Member

Verified: the tunnel client is created in routed mode:

option dev_type 'tun'
option dev 'tuncs1'

@gsanchietti gsanchietti added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Sep 12, 2024
@github-project-automation github-project-automation bot moved this from In progress 🛠 to Done ✅ in NethSecurity Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified All test cases were verified successfully
Projects
Archived in project
Development

No branches or pull requests

3 participants