Opened 10 months ago

Last modified 3 months ago

#28694 new enhancement

When CircuitPadding is implemented in Tor, set it to 0 in sbws

Reported by: teor Owned by:
Priority: Medium Milestone: sbws: 1.1.x-final
Component: Core Tor/sbws Version:
Severity: Normal Keywords: wtf-pad, tor-relay, tor-cell, padding, changes-version-patch, sbws-11x-final-removed-20190312, scanner
Cc: nickm, juga, teor Actual Points:
Parent ID: #29954 Points:
Reviewer: Sponsor: Sponsor2

Description

In #28693, we will implement a tor option for circuit padding.
When that option is implemented, we should set it to 0 in sbws.

Child Tickets

Change History (15)

comment:1 Changed 9 months ago by nickm

Parent ID: #28693

Unparenting this ticket

comment:2 Changed 9 months ago by teor

Parent ID: #28693

sbws doesn't want padding on its circuits, because padding interferes with bandwidth measurements

comment:3 Changed 8 months ago by mikeperry

Take some wtf-pad stuff out of 041-proposed

comment:4 Changed 7 months ago by juga

Milestone: sbws: 1.0.x-finalsbws: 1.1.x-final

Moving to 1.1 milestone since CircuitPadding is not implemented yet

comment:5 Changed 7 months ago by juga

Keywords: changes-version-patch added

Add keyword to tickets that imply a change of version patch, since it might affect the results, but in a backwards compatible way and doesn't add functionality.

comment:6 Changed 7 months ago by juga

Keywords: sbws-11x-final-removed-20190312 added

Move tickets that can wait to next month

comment:7 Changed 7 months ago by juga

Milestone: sbws: 1.1.x-finalsbws: unspecified

Actually remove them from milestone

comment:8 Changed 7 months ago by juga

Keywords: scanner added
Parent ID: #28693#29954

comment:9 Changed 6 months ago by teor

Sponsor: Sponsor2

Move all WTF-PAD tickets that do not have a sponsor into sponsor2.

comment:10 Changed 6 months ago by teor

Keywords: 041-must added

comment:11 Changed 6 months ago by teor

Milestone: sbws: unspecifiedsbws: 1.1.x-final

Bugfix on 1.1.

comment:12 Changed 5 months ago by mikeperry

The CircuitPadding torrc option has been merged to master, FYI (#28693). This can be implemented once swbs start using 0.4.1.

The actual padding machines we're implementing (#28634) only pad HS circuits, though, which sbws does not use. So this is not essential yet, even when 0.4.1 is stable.

comment:13 in reply to:  12 Changed 5 months ago by teor

Replying to mikeperry:

The CircuitPadding torrc option has been merged to master, FYI (#28693). This can be implemented once swbs start using 0.4.1.

sbws uses the installed Tor version. So we should add the CircuitPadding option to sbws. But put it in the list of options that might fail at runtime.

The actual padding machines we're implementing (#28634) only pad HS circuits, though, which sbws does not use. So this is not essential yet, even when 0.4.1 is stable.

Thanks, that's useful for us to know.

comment:14 Changed 5 months ago by teor

Keywords: 041-should added; 041-must removed

We won't block on this ticket, because padding doesn't affect sbws circuits yet.
But we should keep the sbws options up to date with tor, because that's an sbws best practice.

comment:15 Changed 3 months ago by nickm

Keywords: 041-should removed
Note: See TracTickets for help on using tickets.