Opened 10 years ago

Closed 9 years ago

Last modified 8 years ago

#1756 closed task (implemented)

Microdescriptors: clients can fetch and store microdesc consensuses and microdescs

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Deliverable-May2011
Component: Core Tor/Tor Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: #1748 Points:
Reviewer: Sponsor:


To implement microdescriptors (see proposals 158 and 162), we'll need clients to fetch and store microdescriptors and microdescriptor consensuses. The code here will be almost completely the same as that used by caches; see tickets #1754 and #1755.

Child Tickets

Change History (8)

comment:1 Changed 10 years ago by nickm

Status: newneeds_review

Implemented; needs review. See my "microdesc-use" branch.

Incidentally, we may want to hold off merging this till just after is out, so that servers have a chance to get onto before clients start looking for such servers to give them microdescriptors.

Some do-they-matter issues:

  • Does use of microdescriptors or lack thereof leak whether you are a bridge?

comment:2 Changed 9 years ago by arma

Milestone: Deliverable-Sep2010Deliverable-May2011
Parent ID: #1748

comment:3 Changed 9 years ago by nickm

See rebased microdesc-use-v2 branch.

comment:4 Changed 9 years ago by rransom

From commit c68ee71d6baab32744b9eab8d5208e79d556bcfd:

+    if (strcmp(c->value, "auto"))
+      *(int *)lvalue = -1;

Do you need a ! there?

comment:5 Changed 9 years ago by rransom

The automated renames and AUTOBOOL configuration option type could use changes/ files.

comment:6 Changed 9 years ago by nickm

Agree on needing a ! there (and on the next two tests as well), disagree on the changes files: IMO they are neither user-visible enough to be called a feature (we already had tristate options), nor so big a refactoring that they want a "code simplifications and refactorings" section.

Pushed a new microdesc-use-v2 branch with the ! fix.

comment:7 Changed 9 years ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Merged; closing.

comment:8 Changed 8 years ago by nickm

Component: Tor ClientTor
Note: See TracTickets for help on using tickets.