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

Firewall: wrong multiwan status in Analysis page #6593

Closed
andre8244 opened this issue Nov 3, 2021 · 4 comments
Closed

Firewall: wrong multiwan status in Analysis page #6593

andre8244 opened this issue Nov 3, 2021 · 4 comments
Labels
bug A defect of the software verified All test cases were verified successfully

Comments

@andre8244
Copy link
Contributor

Steps to reproduce

  • Configure two red interfaces
  • Disable one of the two red interfaces
  • Go to Analysis page

Expected behavior

  • Multiwan status shows a warning icon

Actual behavior

  • Multiwan status shows an error icon. Error icon should be displayed only if all red interfaces are down

Components

nethserver-firewall-base
@andre8244 andre8244 added the bug A defect of the software label Nov 3, 2021
@andre8244 andre8244 self-assigned this Nov 3, 2021
andre8244 added a commit to andre8244/nethserver-firewall-base that referenced this issue Nov 3, 2021
@andre8244
analysis api. fix multiwan status
611d9df 
NethServer/dev#6593
@andre8244 andre8244 mentioned this issue Nov 3, 2021
Merged
andre8244 added a commit to andre8244/nethserver-firewall-base that referenced this issue Nov 3, 2021
@andre8244
analysis ui. fix interface down icon
6cb73ef 
NethServer/dev#6593
gsanchietti added a commit to NethServer/nethserver-firewall-base that referenced this issue Nov 3, 2021
@gsanchietti
Merge pull request #211 from andre8244/fix-multiwan-status
d7dd091 
NethServer/dev#6593
@nethbot
Copy link
Member

nethbot commented Nov 3, 2021

in 7.9.2009/testing:

@gsanchietti
Copy link
Member

gsanchietti commented Nov 3, 2021 •
edited

Test case
Check the issue is not reproducible

@gsanchietti gsanchietti added the testing Packages are available from testing repositories label Nov 3, 2021
@gsanchietti
Copy link
Member

All WAN's up:

# echo '{"action":"service","service":"multiwan"}' | /usr/libexec/nethserver/api/nethserver-firewall-base/troubleshooting/read | jq -r .status
running

Disable a WAN:

# shorewall disable eth0
Provider red2 (2) stopped

Verify the status:

# echo '{"action":"service","service":"multiwan"}' | /usr/libexec/nethserver/api/nethserver-firewall-base/troubleshooting/read | jq -r .status
warning

Verified

@gsanchietti gsanchietti added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Nov 4, 2021
@nethbot
Copy link
Member

nethbot commented Nov 4, 2021

in 7.9.2009/updates:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A defect of the software verified All test cases were verified successfully
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gsanchietti @andre8244 @nethbot