Opened 7 months ago

Closed 3 months ago

Last modified 2 months ago

#26770 closed enhancement (implemented)

Implement proposal 293: "Other ways for relays to know when to publish"

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: prop293, prop275, 040-roadmap-proposed, StaleDesc
Cc: arma Actual Points: .1
Parent ID: #21642 Points:
Reviewer: teor Sponsor: Sponsor8-can

Description


Child Tickets

Change History (15)

comment:1 Changed 7 months ago by nickm

If we implement this in 0.3.5, we can implement prop275 later down the road and reduce our consensus diff download size by a pretty large amount.

comment:2 Changed 4 months ago by teor

Type: defectenhancement

comment:3 Changed 4 months ago by teor

Keywords: 036-roadmap-proposed added; 035-roadmap-proposed removed

Move likely enhancements from 035-roadmap-proposed to 036-roadmap-proposed

comment:4 Changed 3 months ago by teor

Keywords: 040-roadmap-proposed added; 036-roadmap-proposed removed

0.3.6 is now 0.4.0: changing roadmap keywords

comment:5 Changed 3 months ago by nickm

Owner: set to nickm
Status: newaccepted

comment:6 Changed 3 months ago by nickm

Milestone: Tor: unspecifiedTor: 0.4.0.x-final
Status: acceptedneeds_review

See my branch prop293 with PR at https://github.com/torproject/tor/pull/542 .

comment:7 Changed 3 months ago by nickm

Actual Points: .1

comment:8 Changed 3 months ago by dgoulet

Reviewer: teor

comment:9 Changed 3 months ago by teor

Status: needs_reviewneeds_revision

Please see my review on the pull request: my main questions are about testing, and keeping flags in sync as we add new flags.

comment:10 Changed 3 months ago by nickm

Status: needs_revisionneeds_review

Hi! I've answered your questions, made some changes, and written some tests!

comment:11 Changed 3 months ago by nickm

Status: needs_reviewneeds_revision

Ooh, CI is busted. I need to fix this before it's ready for review.

comment:12 Changed 3 months ago by nickm

Status: needs_revisionneeds_review

Okay, should be fixed and ready for review.

comment:13 Changed 3 months ago by teor

Status: needs_reviewmerge_ready

Looks good to me, I'll leave it to you to open a ticket for the extra tests.

comment:14 Changed 3 months ago by nickm

Keywords: StaleDesc added
Resolution: implemented
Status: merge_readyclosed

#28739 is the ticket for the extra tests.

Main branch squashed and merged.

d14164d8b9d7bf merges the proposal's changes into dir-spec.txt.

comment:15 Changed 2 months ago by nickm

Sponsor: Sponsor8-can
Note: See TracTickets for help on using tickets.