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

Monitoring: error when OpenVPN RoadWarrior has an incomplete configuration #943

Closed
Tbaile opened this issue Nov 29, 2024 · 3 comments
Closed

Monitoring: error when OpenVPN RoadWarrior has an incomplete configuration #943

Tbaile opened this issue Nov 29, 2024 · 3 comments
Labels
verified All test cases were verified successfully
Milestone
NethSecurity 8.4

Comments

@Tbaile
Copy link
Contributor

Tbaile commented Nov 29, 2024 •
edited by gsanchietti
Loading

OpenVPN RoadWarrior creates a "dummy" server even if you dismiss it's initial setup.

Steps to reproduce

  • Go to OpenVPN Road Warrior
  • Click the provided Create Server button
  • Dismiss the form of the sidebar with either Cancel or by pressing ESC
  • Navigate under Monitoring -> Real time monitor -> VPN

Expected behavior

No OpenVPN server should appear.

Actual behavior

Issues arise due to the backend creating a "placeholder" server but all APIs give errors due to missing data:

  • ns.ovpnrw list-users
  • ns.report ovpnrw-bytes-by-hour
  • ns.report ovpnrw-clients-by-day
  • ns.report ovpnrw-count-clients-by-hour

Components

Image version: 8-23.05.5-ns.1.3.0

See also

Ref:

Thanks to mz05er
@Tbaile Tbaile added this to the NethSecurity 8.4 milestone Nov 29, 2024
@github-project-automation github-project-automation bot moved this to ToDo 🕐 in NethSecurity Nov 29, 2024
@Tbaile Tbaile changed the title OpenVPN creates a "ghost" even if you dismiss it's initial setup OpenVPN RoadWarrior creates a "dummy" server even if you dismiss it's initial setup Nov 29, 2024
@andre8244 andre8244 self-assigned this Nov 29, 2024
This was referenced Nov 29, 2024
Closed
Merged
Merged
@gsanchietti gsanchietti changed the title OpenVPN RoadWarrior creates a "dummy" server even if you dismiss it's initial setup Monitoring: error when OpenVPN RoadWarrior has an incomplete configuration Dec 4, 2024
@gsanchietti gsanchietti moved this from ToDo 🕐 to In Progress 🛠 in NethSecurity Dec 4, 2024
gsanchietti pushed a commit that referenced this issue Dec 4, 2024
@andre8244
fix(ns-api): fix ns.ovpnrw list-users (#950)
9d3d217 
Return an empty list if the ovpn instance exists but has not been
configured. Before this change list-users threw an exception in this
scenario

#943
gsanchietti pushed a commit to NethServer/nethsecurity-ui that referenced this issue Dec 4, 2024
@andre8244
fix(monitoring): handle ovpn server not configured (#453)
3b457c1 
NethServer/nethsecurity#943
gsanchietti added a commit to NethServer/nethsecurity-ui that referenced this issue Dec 4, 2024
@gsanchietti
chore(main): release 1.15.1 (#454)
5218f40 
NethServer/nethsecurity#943
gsanchietti added a commit that referenced this issue Dec 4, 2024
@github-actions @gsanchietti
build(deps): ns-ui package: update ns-ui package (automated) (#951)
a2de0d8 
#943 

Co-authored-by: gsanchietti <gsanchietti@users.noreply.github.com>
@gsanchietti
Copy link
Member

Testing image: 23.05.5-ns.1.3.0-93-g225c40bf

Test case

Check the issue is not reproducible

@gsanchietti gsanchietti added the testing Packages are available from testing repositories label Dec 5, 2024
@nethbot nethbot moved this from In Progress 🛠 to Testing in NethSecurity Dec 5, 2024
@mamengoni mamengoni self-assigned this Dec 9, 2024
@mamengoni
Copy link

Tested and verified.

Image

@mamengoni mamengoni added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Dec 9, 2024
@mamengoni mamengoni removed their assignment Dec 9, 2024
@nethbot nethbot moved this from Testing to Verified in NethSecurity Dec 9, 2024
@Tbaile Tbaile mentioned this issue Dec 13, 2024
Merged
@gsanchietti
Copy link
Member

Released on 23.05.5-ns.1.4.0

@github-project-automation github-project-automation bot moved this from Verified to Done ✅ in NethSecurity Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
verified All test cases were verified successfully
Projects
NethSecurity
Archived in project
Milestone
NethSecurity 8.4
Development

No branches or pull requests
4 participants
@gsanchietti @andre8244 @Tbaile @mamengoni