diff options
author | David Goulet <dgoulet@torproject.org> | 2021-10-06 15:41:12 -0400 |
---|---|---|
committer | David Goulet <dgoulet@torproject.org> | 2021-10-06 15:41:12 -0400 |
commit | 065ebd10c2baee20f2963ac7116701d9593027a8 (patch) | |
tree | db390c18805e2867fd3169f7ba3dcdac069d7cba | |
parent | a53c949dcf2816f6878209788c8768b7ca35dc35 (diff) | |
parent | 9348b1b440ad17b3a5dda52b5389bbaf6f23ed56 (diff) | |
download | tor-065ebd10c2baee20f2963ac7116701d9593027a8.tar.gz tor-065ebd10c2baee20f2963ac7116701d9593027a8.zip |
Merge branch 'tor-gitlab/mr/393' into maint-0.4.5
-rw-r--r-- | changes/bug40175 | 5 | ||||
-rw-r--r-- | src/feature/dirclient/dirclient.c | 4 |
2 files changed, 8 insertions, 1 deletions
diff --git a/changes/bug40175 b/changes/bug40175 new file mode 100644 index 0000000000..aa2ce9566f --- /dev/null +++ b/changes/bug40175 @@ -0,0 +1,5 @@ + o Minor features (logging, diagnostic): + - Log decompression failures at a higher severity level, since they + can help provide missing context for other warning messages. + We rate-limit these messages, to avoid flooding the logs if they + begin to occur frequently. Closes ticket 40175. diff --git a/src/feature/dirclient/dirclient.c b/src/feature/dirclient/dirclient.c index f2e1e5b5ff..170802173d 100644 --- a/src/feature/dirclient/dirclient.c +++ b/src/feature/dirclient/dirclient.c @@ -1959,7 +1959,9 @@ dir_client_decompress_response_body(char **bodyp, size_t *bodylenp, /* If we're pretty sure that we have a compressed directory, and * we didn't manage to uncompress it, then warn and bail. */ if (!plausible && !new_body) { - log_fn(LOG_PROTOCOL_WARN, LD_HTTP, + const int LOG_INTERVAL = 3600; + static ratelim_t warning_limit = RATELIM_INIT(LOG_INTERVAL); + log_fn_ratelim(&warning_limit, LOG_WARN, LD_HTTP, "Unable to decompress HTTP body (tried %s%s%s, on %s).", description1, tried_both?" and ":"", |