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.

Sign up for GitHub

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

Jump to bottom

OpenVPN tunnel client: device always forced to "tap" #743

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

OpenVPN tunnel client: device always forced to "tap" #743

cotosso opened this issue Sep 10, 2024 · 2 comments
Assignees
@Tbaile
Labels
verified All test cases were verified successfully
Milestone
NethSecurity 8.3

Comments

@cotosso
Copy link
Contributor

cotosso commented Sep 10, 2024 •
edited
Loading

Steps to reproduce

  • Import an Openvpn tunnel client or configure a new one from scratch.
  • Edit configuration from web UI
  • open Advanced settings

image

  • select Routed Mode

Expected behavior

Tunnel configuration is created accordingly in the database:

option dev_type tun
option dev tun$NAME

Actual behavior
The dev type is always set to tap instead of tun and the dev name is accordingly created (tap$NAME) regardless to the chosen mode (Bridged or Routed).

IMPORTANT
It only happens if Advanced settings section has been opened, otherwise default tun is used.

Components

NethSecurity 8 23.05.4-ns.1.2.0-25-g2a6b69cd
@cotosso cotosso added the bug label Sep 10, 2024
@cotosso cotosso mentioned this issue Sep 10, 2024
Closed
@gsanchietti gsanchietti moved this to Ready ⏯ in NethSecurity Sep 10, 2024
@gsanchietti gsanchietti added this to the NethSecurity 8.3 milestone Sep 10, 2024
@Tbaile Tbaile mentioned this issue Sep 10, 2024
Merged
@Tbaile
Copy link
Contributor

Tbaile commented Sep 11, 2024

Testing image: https://github.com/NethServer/nethsecurity/actions/runs/10812881491/artifacts/1919729133

@Tbaile Tbaile added the testing Packages are available from testing repositories label Sep 11, 2024
@cotosso
Copy link
Contributor Author

cotosso commented Sep 11, 2024

I tested the image:
The proposed fix solved the problem, now configuration file is written accordingly to what is present in the web UI.

@cotosso cotosso added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Sep 11, 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
Assignees

@Tbaile Tbaile

Labels
verified All test cases were verified successfully
Projects
NethSecurity
Archived in project
Milestone
NethSecurity 8.3
Development

No branches or pull requests
3 participants
@gsanchietti @cotosso @Tbaile