Opened 2 years ago

Closed 21 months ago

#24023 closed defect (implemented)

Is proto required for alternate relay implementations?

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: easy doc tor-spec
Cc: Actual Points:
Parent ID: Points: 0.1
Reviewer: Sponsor:

Description

Mike reports on tor-relays that authorities treat the "proto" line as mandatory for non-Tor implementations.

We should document that here:

https://gitweb.torproject.org/torspec.git/tree/dir-spec.txt#n773

Child Tickets

Change History (8)

comment:1 Changed 2 years ago by arma

What do we mean by mandatory? That is, what fails?

comment:2 in reply to:  1 Changed 2 years ago by teor

Replying to arma:

What do we mean by mandatory? That is, what fails?

According to Mike, the relay is not included in the consensus unless it has a "proto" line.

comment:3 Changed 2 years ago by nickm

Keywords: tor-spec added

So, it's not actually mandatory: Tor versions before 0.2.9 didn't have it, and they are still listed in the consensus. Possibly, it is mandatory when the platform line is empty? We should investigate.

comment:4 Changed 2 years ago by nickm

Ah, that appears to be what Mike said.

comment:5 Changed 2 years ago by teor

Milestone: Tor: 0.3.2.x-finalTor: 0.3.3.x-final
Owner: teor deleted

We don't have to do this in 0.3.2

comment:6 Changed 23 months ago by teor

Status: assignednew

Mark all tickets that are assigned to nobody as "new".

comment:7 Changed 21 months ago by nickm

See also #24245.

comment:8 Changed 21 months ago by nickm

Resolution: implemented
Status: newclosed

e6c3257b065f7fe816843bf37de3c3b958d30ee7 in torspec to document that the field will become mandatory.

Note: See TracTickets for help on using tickets.