#25869 closed enhancement (implemented)

Bandwidth List Format

Reported by: juga Owned by:
Priority: Medium Milestone: Tor: 0.3.4.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: bandwidth, bwauth, scanner, torflow
Cc: Actual Points:
Parent ID: #3723 Points:
Reviewer: nickm Sponsor:

Description

See tor-dev. Corrected draft soon.

Child Tickets

Change History (10)

comment:1 Changed 18 months ago by teor

Milestone: Tor: 0.3.4.x-final
Status: newneeds_revision

We are working on this spec here:
https://github.com/torproject/torspec/pull/4

It just needs one final revision, then it will be ready for review again.

Since it's a spec for an existing format, and it's being done by volunteers, I'm going to put it in 0.3.4.

comment:2 Changed 18 months ago by teor

Parent ID: #3723

We need to finalise this proposal before we add the header to the votes.

comment:3 Changed 18 months ago by juga

Status: needs_revisionneeds_review

First revision being send to tor-dev@.
Work moved to https://github.com/juga0/torspec/commits/bandwidth-file-spec, as the previous branch was missing the first draft sent to tor-dev@

comment:4 Changed 18 months ago by dgoulet

Reviewer: nickm

comment:5 Changed 18 months ago by nickm

Status: needs_reviewneeds_revision

Hi! I posted a review to tor-dev at

https://lists.torproject.org/pipermail/tor-dev/2018-May/013141.html

Good work!

comment:6 Changed 18 months ago by juga

nickm, teor, would you mind to review/answer all the questions i left in [NOTE] (same branch) before i send again to tor-dev@?.

As suggested by teor in tor-dev@ and https://trac.torproject.org/projects/tor/ticket/25960#comment:9, i would create other file with for version 2.0.0 (replacing key_value format with SP). Same of the questions would still apply.

comment:7 Changed 18 months ago by juga

Status: needs_revisionneeds_review

comment:8 Changed 18 months ago by nickm

Status: needs_reviewneeds_revision

Done via personal email this morning -- please let me know if there's more to do!

comment:9 Changed 17 months ago by juga

Status: needs_revisionneeds_review
Summary: Proposal: Bandwidth Measurements Document FormatBandwidth List Format

comment:10 Changed 17 months ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Okay -- I've squashed that branch down to a single commit, and merged it as 9465f9c0713b21. There may be more issues to discuss about the format, but let's take this spec as our starting point for those.

Note: See TracTickets for help on using tickets.