Skip to content

Commit

Permalink
Merge pull request ansible#5162 from JensRantil/fix-corrupt-module-ou…
Browse files Browse the repository at this point in the history
…tputs

More don't catch `SystemExit`
  • Loading branch information
mpdehaan authored and jctanner committed Dec 19, 2013
1 parent 516d928 commit 7d209ed
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion library/monitoring/nagios
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ def main():
m = int(minutes)
if not isinstance(m, types.IntType):
module.fail_json(msg='minutes must be a number')
except:
except Exception:
module.fail_json(msg='invalid entry for minutes')

##################################################################
Expand Down
2 changes: 1 addition & 1 deletion library/packaging/apt_key
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ def download_key(module, url):
module.fail_json("error connecting to download key from url")
data = connection.read()
return data
except:
except Exception:
module.fail_json(msg="error getting key id from url", traceback=format_exc())


Expand Down

0 comments on commit 7d209ed

Please sign in to comment.