Strip && and || from query for alerts in polling #2476

Merged
merged 1 commit into from Nov 21, 2015

Projects

None yet

3 participants

@laf
Member
laf commented Nov 21, 2015

Fix #2468

&& and || is stripped in WebUI but not in the polling code. Updated now to make it work in both areas.

@laf laf Strip && and || from query for alerts in polling
d153cf4
@f0o
Member
f0o commented Nov 21, 2015

Irc please

On 21 November 2015 11:14:37 CET, LibreNMS-CI notifications@github.com wrote:

Auto-Deploy finished, Test PR at http://2476.ci.librenms.org or
https://2476.ci.librenms.org


Reply to this email directly or view it on GitHub:
#2476 (comment)

Sent from my Android device with K-9 Mail. Please excuse my brevity.

@f0o f0o merged commit 91f825e into librenms:master Nov 21, 2015

3 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer Analysis No new issues
Details
Scrutinizer Tests Tests are not configured
@laf laf deleted the laf:device-groups branch Nov 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment