fix: Cisco remote access stats bugfix #4293 #4309

Merged
merged 1 commit into from Sep 2, 2016

Projects

None yet

3 participants

@Rosiak
Contributor
Rosiak commented Sep 1, 2016

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

Fix #4293
Fixes the following:

  • Issue in the polling, typo(crasWebvpnNumSessions).
  • Issue in the polling, the check didn't take into account if all values returned 0(false), also didn't take into account that any of the values could be bogus.
  • Issue in the graphing, no reason to use the webvpn_only calculation when the values returned are perfectly fine.

Also a small visual update.

@Rosiak Rosiak fix: Cisco remote access stats bugfix #4293
98cefea
@murrant murrant merged commit b81e67c into librenms:master Sep 2, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@deutor deutor added a commit to deutor/librenms that referenced this pull request Sep 6, 2016
@Rosiak @deutor Rosiak + deutor fix: Cisco remote access stats bugfix #4293 (#4309) 0f0a65d
@Rosiak Rosiak deleted the Rosiak:issue-4293 branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment