]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
Upgrade and rate-limit compression failure message.
authorNick Mathewson <nickm@torproject.org>
Thu, 27 May 2021 14:49:37 +0000 (10:49 -0400)
committerNick Mathewson <nickm@torproject.org>
Thu, 27 May 2021 14:49:37 +0000 (10:49 -0400)
Without this message getting logged at 'WARN', it's hard to
contextualize the messages we get about compression bombs, so this
message should fix #40175.

I'm rate-limiting this, however, since it _could_ get spammy if
somebody on the network starts acting up.  (Right now it should be
very quiet; I've asked Sebastian to check it, and he says that he
doesn't hit this message in practice.)

Closes #40175.

src/feature/dirclient/dirclient.c

index a5dd856729bcccbd4ace83d0a9a718368caa953b..b589e829f985dbbee1ba76eb21c0831546f7aaa1 100644 (file)
@@ -1944,7 +1944,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 ":"",