Feature: Added alerts output to capture system #4574

Merged
merged 5 commits into from Sep 26, 2016

Projects

None yet

4 participants

@laf
Member
laf commented Sep 24, 2016

Please note

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

Added alerts to the capture system so people can run checks against rules for a device.

image

@laf laf Feature: Added alerts output to capture system
017d2d7
html/includes/functions.inc.php
+ * @param $filename
+ * @param $output
+ */
+function file_download($filename, $output)
@murrant
murrant Sep 24, 2016 Contributor

I'd rename this variable to $content

@laf
laf Sep 24, 2016 Member

Updated and pushed.

@laf laf Renamed $output to $content
b0746a4
@laf laf Rebased
2471f4d
html/includes/functions.inc.php
@@ -1343,3 +1343,41 @@ function ipmiSensorName($hardwareId, $sensorIpmi, $rewriteArray)
return $sensorIpmi;
}
}
+
+function get_auth_ad_user_filter($username)
@murrant
murrant Sep 26, 2016 Contributor

Something wrong here, these shouldn't be here...

laf added some commits Sep 26, 2016
@laf laf Merge branch 'master' of github.com:librenms/librenms into alerts-cap…
…ture
18f6974
@laf laf removed duplicate functions
e052929
@laf
Member
laf commented Sep 26, 2016

Updated.

@scrutinizer-notifier

The inspection completed: 1 updated code elements

@murrant murrant merged commit 9936aa7 into librenms:master Sep 26, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:alerts-capture branch Sep 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment