Opened 6 years ago

Closed 6 years ago

#9922 closed defect (fixed)

We declare bootstrapping at 50% when we get our bridge descriptor

Reported by: arma Owned by:
Priority: Medium Milestone: Tor: 0.2.5.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: tor-client
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

In load_downloaded_routers() we end with

  control_event_bootstrap(BOOTSTRAP_STATUS_LOADING_DESCRIPTORS,
                          count_loading_descriptors_progress());

and we do that even if it's just because we fetched the bridge descriptor from a bridge.

That means we miss out on learning later whether we could fetch the consensus.

Child Tickets

Change History (5)

comment:1 Changed 6 years ago by karsten

Is this related to #9229?

comment:2 in reply to:  1 Changed 6 years ago by arma

Replying to karsten:

Is this related to #9229?

It's related, but not the same. This one is just about log messages (or lack thereof) and the bootstrapping control events that go with them.

comment:3 Changed 6 years ago by arma

bug9922 in my arma fixes it for me.

it also has a partial changelog file which somebody should flesh out

comment:4 Changed 6 years ago by arma

Status: newneeds_review

comment:5 Changed 6 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Looks plausible to me; I've written the changelog and merged it into master.

Note: See TracTickets for help on using tickets.