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

Cockpit: Web proxy status unavailable #6185

Closed
filippocarletti opened this issue May 28, 2020 · 7 comments
Closed

Cockpit: Web proxy status unavailable #6185

filippocarletti opened this issue May 28, 2020 · 7 comments
Labels
bug A defect of the software verified All test cases were verified successfully

Comments

@filippocarletti
Copy link
Member

filippocarletti commented May 28, 2020 •
edited

The Cockpit dashboard page of the Web proxy shows a forever-running spinner in the left top box instead of the status if the web filter app is not installed.

Steps to reproduce

  • In the Software Center, install only the Web proxy, do NOT install the Web filter
  • Go to the web proxy app

Expected behavior

Status of the proxy

Actual behavior

Spinner

Workaround

Install web filter
@filippocarletti filippocarletti added the bug A defect of the software label May 28, 2020
@filippocarletti
Copy link
Member Author

API output:

[root@ns7 ~]# echo '{"action":"status","hostname":"ns7.neth.net"}' | /usr/bin/sudo /usr/libexec/nethserver/api/nethserver-squid/dashboard/read 
Argument "0.048737%" isn't numeric in sprintf at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 122.
Can't call method "get_all" on an undefined value at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 126.

The problem is on line 126.
Line 122 is about cpu usage.

@filippocarletti
Copy link
Member Author

API output from a different system, with more cosmetic problems:

[root@nsec-primary ~]# echo '{"action":"status","hostname":"ns7.neth.net"}' | /usr/bin/sudo /usr/libexec/nethserver/api/nethserver-squid/dashboard/read | jq
Argument "0.906786%" isn't numeric in sprintf at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 121.
Use of uninitialized value $timestamp in numeric gt (>) at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 140.
Use of uninitialized value $av_db in numeric gt (>) at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 140.
Use of uninitialized value $timestamp in numeric gt (>) at /usr/libexec/nethserver/api/nethserver-squid/dashboard/read line 140.

@andre8244 andre8244 self-assigned this May 29, 2020
andre8244 added a commit to andre8244/nethserver-squid that referenced this issue May 29, 2020
@andre8244
api & ui: fixed dashboard and filter pages
1906364 
- Dashboard: fixed proxy status
- Filter: fixed UI when filter packages are not installed

NethServer/dev#6185
@andre8244 andre8244 mentioned this issue May 29, 2020
Merged
gsanchietti pushed a commit to NethServer/nethserver-squid that referenced this issue Jun 3, 2020
@andre8244
Fixed dashboard and filter pages (#56)
97d388e 
- Dashboard: fixed proxy status
- Filter: fixed UI when filter packages are not installed

NethServer/dev#6185
@nethbot
Copy link
Member

nethbot commented Jun 3, 2020

in 7.8.2003/testing:

@andre8244 andre8244 added the testing Packages are available from testing repositories label Jun 3, 2020
@andre8244 andre8244 removed their assignment Jun 3, 2020
@andre8244
Copy link
Contributor

Test case 1 (update)

  • Update nethserver-squid with testing rpm
  • Verify that bug is no more reproducible

@andre8244
Copy link
Contributor

Test case 2 (fresh install)

  • Install nethserver-squid testing rpm
  • Access Web Proxy & Filter app on Cockpit and verify that dashboard loads correctly (no spinning loader)
  • Go to Filter page and verify that Missing packages message appears
  • Click Install packages button to install nethserver-squidguard and nethserver-squidclamav packages
  • Verify that Filter page now displays Categories not yet downloaded message
  • Configure web proxy and filter and verify that dashboard loads correctly and shows proxy and filter status as expected

@gsanchietti gsanchietti self-assigned this Jun 3, 2020
@gsanchietti
Copy link
Member

Both test cases have been verified.
If the filter is not installed, the UI asks to install it.

@gsanchietti gsanchietti removed their assignment Jun 3, 2020
@gsanchietti gsanchietti added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Jun 3, 2020
@nethbot
Copy link
Member

nethbot commented Jun 3, 2020

in 7.8.2003/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
4 participants
@filippocarletti @gsanchietti @andre8244 @nethbot