Rip out AUTHDIR_NEWDESCS event?

I vaguely remember that weasel added this feature back when the directory authorities were running the Naming scripts?

Now all of those scripts are gone.

I don't know of any directory authorities who run with their control port open now.

Maybe this is a great time to just get rid of it?

Motivated by the bug in #22371 (moved).

changed milestone to %Tor: 0.3.2.x-final

added component::core tor/tor milestone::Tor: 0.3.2.x-final owner::nickm priority::medium resolution::implemented review-group-23 reviewer::dgoulet severity::normal status::closed type::enhancement labels

(Maybe the test network, or chutney, would still enjoy having it though?)

Replying to arma:

(Maybe the test network, or chutney, would still enjoy having it though?)

If I were testing relay descriptor checks in chutney, I'd want it.

weasel confirms that he doesn't use it anymore.

So I think no dir auths on the main network have a control port open.

We could keep the feature if we think maintaining it for chutney etc is sufficiently useful.

I don't use it.

Damian, would anything go wrong if we deprecated this and/or just took it out?

Trac:
Cc: N/A to atagar

Hi Nick. Nope, feel free to deprecate and/or remove. That said, I'd appreciate a note saying the tor version where it's gone (stem makes an effort to provide nice exception messages when an event doesn't exist due to your version).

Okay, let's do this in 0.3.2.x then. But if we decide not to, we should reopen and possibly merge #22371 (moved) .

Trac:
Owner: N/A to nickm
Status: new to accepted

See branch ticket22377 in my public Tor repository.

See also branch ticket22377_spec in my torspec repository.

Trac:
Status: accepted to needs_review

Put 0.3.2 needs_review and merge_ready tickets into review-group-23.

Trac:
Keywords: N/A deleted, review-group-23 added

Code lgtm;

About the spec patch, shouldn't we leave section 4.1.8. intact but mention at the beginning that it has been removed? I'm saying that because we have many supported version that still has it and removing it from our spec could lead to some confusion because "oh stem has that but I can't find it in the spec".

We kind of did that with "dir-usage" mentioning that we have removed it in 029.

Moving that in need info but if consensus is that it's fine, merge ready!

Trac:
Status: needs_review to needs_information
Reviewer: N/A to dgoulet

oh, good point. will make that change then merge.

changed, merged. Thanks!

Trac:
Resolution: N/A to implemented
Status: needs_information to closed

closed

moved to tpo/core/tor#22377 (closed)