#31356: 0.4.1 relays should list Padding=2
---------------------------------+------------------------------------
Reporter: mikeperry | Owner: (none)
Type: defect | Status: needs_review
Priority: Medium | Milestone: Tor: 0.4.1.x-final
Component: Core Tor/Tor | Version: Tor: 0.4.0.5
Severity: Normal | Resolution:
Keywords: wtf-pad, 041-should | Actual Points:
Parent ID: | Points: 1
Reviewer: | Sponsor: Sponsor2
---------------------------------+------------------------------------
Old description:
> Somehow we accidentally merged the protover for padding support while
> doing the incremental merge thing, and 0.4.0 relays are advertising
> padding that they don't support. This is mostly harmless, because the
> negotiation will not succeed and then clients will stop, but it will
> result in those clients emitting a "Middle node did not accept our
> padding request" protocol warn/info message.
>
> ~~We should just remove this protover field from 0.4.0.x.~~
>
> At the weekly meeting last week, we decided that we can't remove a
> protover once it's been released.
> Instead, we will:
> * make 0.4.1 and later relays declare Padding=1,2 (pre-0.4.1 stable)
> * make 0.4.1 and later clients require Padding=2 (padding is not on by
> default, so we can do this at any time)
New description:
Somehow we accidentally merged the protover for padding support while
doing the incremental merge thing, and 0.4.0 relays are advertising
padding that they don't support. This is mostly harmless, because the
negotiation will not succeed and then clients will stop, but it will
result in those clients emitting a "Middle node did not accept our padding
request" protocol warn/info message.
~~We should just remove this protover field from 0.4.0.x.~~
At the weekly meeting last week, we decided that we can't remove a
protover once it's been released.
Instead, we will:
* make 0.4.1 and later relays declare Padding=2 (pre-0.4.1 stable)
* make 0.4.1 and later clients require Padding=2 (padding is not on by
default, so we can do this at any time)
Edited to simplify: we don't need to preserve compatibility with alphas.
--
Comment (by teor):
I also edited the ticket description to match our discussion, so the
reviewer is not confused.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/31356#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs
