Opened 11 months ago

Closed 7 months ago

#29203 closed defect (fixed)

Add a way to specify machines as reduced circuit padding

Reported by: mikeperry Owned by:
Priority: Medium Milestone: Tor: 0.4.1.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: wtf-pad
Cc: Actual Points: 0.33
Parent ID: #28634 Points: 4
Reviewer: asn Sponsor: Sponsor2

Description

We need a way for clients to specify that they want reduced circuit padding -- either by only selecting low overhead padding machines, or by requesting different overhead caps for the machines they use.

Child Tickets

Change History (16)

comment:1 Changed 11 months ago by mikeperry

Keywords: 041-proposed added

comment:2 Changed 10 months ago by mikeperry

Keywords: 041-proposed removed

Take some wtf-pad stuff out of 041-proposed

comment:3 Changed 10 months ago by mikeperry

Priority: HighMedium

comment:4 Changed 8 months ago by nickm

Milestone: Tor: 0.4.2.x-final

comment:5 Changed 8 months ago by mikeperry

Status: newneeds_review

https://github.com/torproject/tor/pull/965

That PR contains fixes for #28693, #30173, and #29203. I recommend that the same reviewer review all three tickets -- they are extremely similar and related code (hence all in the same branch with the same test; doing these separetely would involve needless extra mental context switching and extra communication).

comment:6 Changed 8 months ago by mikeperry

Actual Points: 0.33

comment:7 Changed 8 months ago by teor

Status: needs_reviewneeds_revision

Both Travis and Appveyor CIs failed on https://github.com/torproject/tor/pull/965

comment:8 Changed 8 months ago by teor

Milestone: Tor: 0.4.2.x-finalTor: 0.4.1.x-final

These tickets all have code in the same pull request, so I'm assuming they're all 0.4.1.

comment:9 Changed 8 months ago by mikeperry

Status: needs_revisionneeds_review

CI issues fixed in PR. Also spec changes at https://github.com/torproject/torspec/pull/78

comment:10 Changed 8 months ago by asn

Reviewer: asn

comment:11 Changed 8 months ago by asn

Status: needs_reviewneeds_revision

Comment left in #30173.

comment:12 Changed 8 months ago by mikeperry

Status: needs_revisionneeds_review

comment:13 Changed 7 months ago by asn

Status: needs_reviewneeds_revision

comment:14 Changed 7 months ago by mikeperry

Parent ID: #28634
Status: needs_revisionneeds_review

comment:15 Changed 7 months ago by asn

Status: needs_reviewmerge_ready

comment:16 Changed 7 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

merged!

Note: See TracTickets for help on using tickets.