Make hidden service descriptor creation more consistent

This cleans up legacy/trac#21596 (moved): now that circuit_established is reliable, it can be used during descriptor creation as well. This prevents a regression to bugs like legacy/trac#21594 (moved).

changed milestone to %Tor: 0.3.1.x-final in legacy/trac

added actualpoints::0.1 in Legacy / Trac component::core tor/tor in Legacy / Trac milestone::Tor: 0.3.1.x-final in Legacy / Trac parent::21446 in Legacy / Trac points::0.1 in Legacy / Trac priority::medium in Legacy / Trac resolution::fixed in Legacy / Trac severity::normal in Legacy / Trac status::closed in Legacy / Trac tor-hs in Legacy / Trac type::defect in Legacy / Trac version::tor 0.2.7.2-alpha in Legacy / Trac labels

Please see my branch bug21599, which includes the change it depends on from legacy/trac#21596 (moved).

Trac:
Status: new to needs_review
Version: N/A to Tor: 0.2.7.2-alpha

Patch lgtm however it will conflict because that patch also has the change of remove_invalid_intro_points() I've seen in another ticket.

Trac:
Status: needs_review to needs_revision

Replying to dgoulet:

Patch lgtm however it will conflict because that patch also has the change of remove_invalid_intro_points() I've seen in another ticket.

No, it won't conflict with legacy/trac#21596 (moved) - git is smart enough to merge commits that match.

(See my branch today-in-hs for how this works.)

Trac:
Status: needs_revision to merge_ready

merged to master; looks plausible.

Trac:
Resolution: N/A to fixed
Status: merge_ready to closed

closed

changed time estimate to 48m

added 48m of time spent

mentioned in issue legacy/trac#21621 (moved)

mentioned in issue legacy/trac#21622 (moved)

moved from legacy/trac#21599 (moved)

added Bug label and removed 1 deleted label

added 1 deleted label and removed 1 deleted label