Add support for parsing bridge-distribution-request lines in Stem

From #18329 (moved), we added a new bridge-server-descriptor line: bridge-distribution-request, which, if set to "none" should tell BridgeDB not to distribute a bridge. This allows, for example, BridgeDB to avoid distributing the default Tor Browser bridges, and allows other bridges to not be distributed, but to still submit their usage statistics for Metrics.

added component::archived/stem owner::atagar points::1 priority::high resolution::implemented severity::normal status::closed type::enhancement labels

Hi Isis, sorry this went so long without a reply. Been down with the flu.

Took a quick peek at the CollecTor docs but not spotting this new field...

https://collector.torproject.org/#bridge-descriptors

Is it live? Could you please provide a descriptor with examples of this (both the none and the non-none values)? I'd also like the field to be documented before we add it to Stem.

Trac:
Status: new to needs_information

Hi isis, did you still want this? If so then I need some information.

Trac:
Resolution: N/A to user disappeared
Status: needs_information to closed

Hi atagar! There's a spec patch from #18329 (moved) which should be usable to implement this. It's not live in Tor yet, since #18329 (moved) is waiting (among other things) for it to be parseable by BridgeDB (and hence Stem), before merging it since it would be bad if operators thought their bridge was private and it wasn't actually respected.

Trac:
Status: closed to reopened
Resolution: user disappeared to N/A

Ha! So spec patch is awaiting tor which is awaiting bridgedb which is awaiting stem which is awaiting the spec. Thanks, you made me chuckle this morning. :P

No problem, this is a trivial field to add.

Stem support pushed...

https://gitweb.torproject.org/stem.git/commit/?id=3eb5941

No big rush but this will need to be followed up with the tor spec change otherwise I'll revert this in a future revision (I have a 'must be in the spec' policy because Stem's predecessor, TorCtl, accumulated support for fields that never existed which was confusing). Happy to make an exception here, just giving a head's up that this will need to be followed up with the addition on tor's side.

Feel free to reopen if ya need anything else.

Trac:
Status: reopened to closed
Resolution: N/A to implemented

closed

changed time estimate to 8h

mentioned in issue #23957 (moved)

mentioned in issue #23958 (moved)

mentioned in issue tpo/anti-censorship/bridgedb#23957 (closed)