#31356: 0.4.1 relays should list Padding=1,2
---------------------------------+------------------------------------
Reporter: mikeperry | Owner: (none)
Type: defect | Status: needs_revision
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
---------------------------------+------------------------------------
Changes (by teor):
* status: needs_review => needs_revision
* keywords: wtf-pad => wtf-pad, 041-should
* milestone: => Tor: 0.4.1.x-final
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.
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=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)
--
Comment:
At the weekly meeting last week, we decided that we can't remove a
protover once it's been released. See the ticket description for details.
I'll open another ticket for the client change.
Mike, I'm happy to do this change, if that would help you with your work
overload.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/31356#comment:6>
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
