diff options
author | Lamont Granquist <lamont@scriptkiddie.org> | 2019-07-24 15:23:57 -0700 |
---|---|---|
committer | Lamont Granquist <lamont@scriptkiddie.org> | 2019-07-24 18:33:51 -0700 |
commit | 81f8d729edcd5237aaad19c351dae46fd8e7c0d6 (patch) | |
tree | d22cac0e13faa3349e7921a27ee29c1d89266de3 /bin | |
parent | 303282f31ea20ff5dcdefe42e19c0bb8a42f4178 (diff) | |
download | chef-81f8d729edcd5237aaad19c351dae46fd8e7c0d6.tar.gz |
Tweak data collector exception handling
This isn't Java and Net::HTTP can throw a billion things, so just rescue
(nearly) everything. I have a hard time imagining that there's a
failure here that we wouldn't want to ignore, other than the ones like
OOM thrown by Exception that indicate internal failures.
Also only throw the "this is normal" info message on 404s, other
responses like 500s should be warns even if it is set to ignore
failures (people with data collectors correctly setup need to know about
500s even if we keep going, since that is abnormal).
This also fixes a bug in handling exceptions like Timeout::Error which
do not have a response.code at all which would throw NoMethodError.
closes #8749
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Diffstat (limited to 'bin')
0 files changed, 0 insertions, 0 deletions