diff options
author | Brendan Jurd <direvus@gmail.com> | 2014-10-02 16:32:30 +1000 |
---|---|---|
committer | Brendan Jurd <direvus@gmail.com> | 2014-10-02 16:32:30 +1000 |
commit | 6157a6552fa236c92a938e70f975c62cc87ac780 (patch) | |
tree | b5b8e045364976e818215e5761690fe46b433a0a | |
parent | cb69744bcee4b4217d83b4a30006635ba69e2aa0 (diff) | |
download | ansible-modules-core-6157a6552fa236c92a938e70f975c62cc87ac780.tar.gz |
Add word boundary in apache2_module regexp
Add a word boundary \b to the regexp for checking the output of a2{en,dis}mod,
to avoid a false positive for a module that ends with the same text as the
module we're working on.
For example, the previous regexp r'.*spam already enabled' would also match
against 'eggs_spam already enabled'.
Also, get rid of the redundant '.*' from the end of the regexp.
-rw-r--r-- | web_infrastructure/apache2_module.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/web_infrastructure/apache2_module.py b/web_infrastructure/apache2_module.py index 39351482..13b9e821 100644 --- a/web_infrastructure/apache2_module.py +++ b/web_infrastructure/apache2_module.py @@ -51,7 +51,7 @@ def _disable_module(module): a2dismod_binary = module.get_bin_path("a2dismod") result, stdout, stderr = module.run_command("%s %s" % (a2dismod_binary, name)) - if re.match(r'.*' + name + r' already disabled.*', stdout, re.S): + if re.match(r'.*\b' + name + r' already disabled', stdout, re.S): module.exit_json(changed = False, result = "Success") elif result != 0: module.fail_json(msg="Failed to disable module %s: %s" % (name, stdout)) @@ -63,7 +63,7 @@ def _enable_module(module): a2enmod_binary = module.get_bin_path("a2enmod") result, stdout, stderr = module.run_command("%s %s" % (a2enmod_binary, name)) - if re.match(r'.*' + name + r' already enabled.*', stdout, re.S): + if re.match(r'.*\b' + name + r' already enabled', stdout, re.S): module.exit_json(changed = False, result = "Success") elif result != 0: module.fail_json(msg="Failed to enable module %s: %s" % (name, stdout)) |