Module: check_mk
Branch: master
Commit: 0844df449f0445275013d5920898803e132ee57a
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=0844df449f0445…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Thu Apr 12 16:29:16 2012 +0200
Renamed logwatch editor to logwatch analyzer, Made dialog usable without hostname/filename
---
web/htdocs/wato.py | 39 +++++++++++++++++++++------------------
1 files changed, 21 insertions(+), 18 deletions(-)
diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py
index f5d2d70..3833af5 100644
--- a/web/htdocs/wato.py
+++ b/web/htdocs/wato.py
@@ -9913,8 +9913,12 @@ def mode_pattern_editor(phase):
if phase == "title":
if not hostname and not item:
return _("Logfile Pattern Analyzer")
+ elif not hostname:
+ return _("Logfile Patterns of Logfile %s on all Hosts") % (item)
+ elif not item:
+ return _("Logfile Patterns of Host %s") % (hostname)
else:
- return _("Logfile Patterns of Host %s: %s") % (hostname, item)
+ return _("Logfile Patterns of Logfile %s on Host %s") % (item, hostname)
elif phase == "buttons":
html.context_button(_("Home"), make_link([("mode", "main")]), "home")
@@ -9940,24 +9944,8 @@ def mode_pattern_editor(phase):
return
if phase == "action":
- if not hostname:
- raise MKUserError('host', _('The mandatory parameter "host" is missing.'))
-
- if not host:
- raise MKUserError('host', _('The given host does not exist.'))
-
- if not item:
- raise MKUserError('file', _('The mandatory parameter "file" is missing.'))
-
return
- if not host:
- html.add_user_error('host', _('The given host does not exist.'))
- hostname = ''
- else:
- # FIXME: Check if item exists
- pass
-
html.write(
'<div class=info> '
+ _('On this page you can test the defined logfile patterns against a custom text, '
@@ -9995,6 +9983,12 @@ def mode_pattern_editor(phase):
html.hidden_fields()
html.end_form()
+ # Bail out if the given hostname does not exist
+ if hostname and not host:
+ html.add_user_error('host', _('The given host does not exist.'))
+ html.show_user_errors()
+ return
+
varname = 'checkgroup_parameters:logwatch_rules'
rulespec = g_rulespecs[varname]
all_rulesets = load_all_rulesets()
@@ -10027,10 +10021,19 @@ def mode_pattern_editor(phase):
# Check if this rule applies to the given host/service
if hostname:
+ # If hostname (and maybe filename) try match it
reason = rule_matches_host_and_item(
rulespec, tag_specs, host_list, item_list, folder, g_folder, hostname, item)
+ elif item:
+ # If only a filename is given
+ reason = False
+ for i in item_list:
+ if re.match(i, str(item)):
+ reason = True
+ break
else:
- reason = _('No hostname given.')
+ # If no host/file given match all rules
+ reason = True
match_img = ''
if reason == True:
Module: check_mk
Branch: master
Commit: 80594ad3a328c6e8dddc3a75d28df6fdfc81cf08
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=80594ad3a328c6…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Wed Apr 11 11:43:54 2012 +0200
Updated bug entries #0691
---
.bugs/691 | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/.bugs/691 b/.bugs/691
new file mode 100644
index 0000000..0f37013
--- /dev/null
+++ b/.bugs/691
@@ -0,0 +1,9 @@
+Title: Remove/Secure eval() on html.var()
+Component: wato
+State: open
+Date: 2012-04-11 11:39:35
+Targetversion: 1.2.0
+Class: nastiness
+
+It is VERY insecure and VERY easy to do bad things at these places. They must be
+secured/removed in some way.