[tor-bugs] #33072 [Core Tor/Tor]: When under load, give 503 aggressively for dirport requests without compression

[Tor Bug Tracker & Wiki]

#33072: When under load, give 503 aggressively for dirport requests without
compression
-------------------------------------------------+-------------------------
 Reporter:  nickm                                |          Owner:  dgoulet
     Type:  defect                               |         Status:
                                                 |  needs_revision
 Priority:  Medium                               |      Milestone:  Tor:
                                                 |  0.4.4.x-final
Component:  Core Tor/Tor                         |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  network-health 044-should consider-  |  Actual Points:
  backport-after-0434 042-backport 043-backport  |
Parent ID:  #33018                               |         Points:
 Reviewer:  teor                                 |        Sponsor:
-------------------------------------------------+-------------------------
Changes (by teor):

 * status:  needs_review => needs_revision


Comment:

 It looks like the code, function comments, and changes files are out of
 sync in these branches.

 These lines are only deleted in maint-0.4.3:
 {{{
   if (c_method != NO_METHOD) {
       /* Always answer compressed request. */
       return false;
     }
 }}}

 Are there tests that give a different result when this code is deleted?
 (I don't want to block on tests, so maybe we could do them later in
 master.)

 Also, I'm a bit confused about how you want me to merge here. There's no
 forward merges in these PRs. So when we backport, we'll need to do an
 "ours" merge forward, to avoid conflicts. Is that ok?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/33072#comment:25>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online