nihilist/tor
1
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/core/tor.git synced 2024-11-27 22:03:31 +01:00
Code Issues Packages Projects Releases Wiki Activity

remove confusing comment about conditional consensus download

Browse Source 
We removed that feature in 0.2.4.2-alpha, but some comments seem to
have lingered.

I didn't add a changes/ file since this is just internal code cleanup.
This commit is contained in:
Roger Dingledine 2016-10-02 19:13:35 -04:00
parent 53255e925c
commit 9228db5f1b
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/or/directory.c
View File

@ -495,9 +495,6 @@ MOCK_IMPL(void, directory_get_from_dirserver, (
* sort of dir fetch we'll be doing, so it won't return a bridge * sort of dir fetch we'll be doing, so it won't return a bridge
* that can't answer our question. * that can't answer our question.
*/ */
/* XXX+++++ Not all bridges handle conditional consensus downloading,
* so, for now, never assume the server supports that. -PP
* Is that assumption still so in 2016? -NM */
const node_t *node = choose_random_dirguard(type); const node_t *node = choose_random_dirguard(type);
if (node && node->ri) { if (node && node->ri) {
/* every bridge has a routerinfo. */ /* every bridge has a routerinfo. */
@ -1305,9 +1302,9 @@ compare_strs_(const void **a, const void **b)
/** Return the URL we should use for a consensus download. /** Return the URL we should use for a consensus download.
* *
* This url depends on whether or not the server we go to * Use the "conditional consensus downloading" feature described in
* is sufficiently new to support conditional consensus downloading, * dir-spec.txt, i.e.
* i.e. GET .../consensus/<b>fpr</b>+<b>fpr</b>+<b>fpr</b> * GET .../consensus/<b>fpr</b>+<b>fpr</b>+<b>fpr</b>
* *
* If 'resource' is provided, it is the name of a consensus flavor to request. * If 'resource' is provided, it is the name of a consensus flavor to request.
*/ */