Opened 10 months ago

Closed 9 months ago

Last modified 8 months ago

#23898 closed defect (implemented)

tor-spec: move IPv6 addresses from microdescs to microdesc consensus

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: proposal, ipv6, tor-spec, prop283, review-group-26
Cc: Actual Points: 1.0
Parent ID: #20916 Points: 0.5
Reviewer: nickm Sponsor: SponsorV-can

Description

My torspec branch bug20916 documents the changes in #20916 and children. And it fixes #13043.

Child Tickets

TicketStatusOwnerSummaryComponent
#13043closedteortorspec lies about accepting both IPv4 and IPv6 for ORAddress linesCore Tor/Tor

Change History (12)

comment:1 Changed 10 months ago by teor

And it includes a draft proposal for 20916.

comment:2 Changed 10 months ago by teor

Actual Points: 0.51.0
Keywords: proposal added; needs-proposal removed
Status: newneeds_review

This proposal and some updates to dir-spec that document the new consensus methods and the current state of IPv6 ORPort support are in my torspec branch bug23826-23828.

This is the spec for the tor branch in #23826 and #23828.

I'm just about to send it to tor-dev@ for review.

comment:3 Changed 10 months ago by teor

This is under review on tor-dev@ , here is the link:
https://lists.torproject.org/pipermail/tor-dev/2017-October/012518.html

comment:4 Changed 10 months ago by nickm

Keywords: prop283 added

comment:5 Changed 10 months ago by teor

I have rebased this on top of master and #24007 (there were conflicts), and cleanly split out current and future behaviour. See my branch bug23826-23828-v2.

comment:6 Changed 10 months ago by teor

Owner: set to teor
Status: needs_reviewassigned

These are all mine

comment:7 Changed 9 months ago by teor

Status: assignedneeds_review

I updated proposal 283 based on nickm's feedback. See my github branch bug23826-23828-v2.

comment:8 Changed 9 months ago by nickm

Keywords: review-group-26 added

Creating review-group-26.

comment:9 Changed 9 months ago by nickm

Reviewer: nickm

comment:10 Changed 9 months ago by nickm

Status: needs_reviewmerge_ready

This looks okay to me.

comment:11 Changed 9 months ago by nickm

Resolution: implemented
Status: merge_readyclosed

Merged to torspec.

comment:12 Changed 8 months ago by teor

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