Opened 5 years ago

Closed 5 years ago

#15013 closed enhancement (fixed)

Add new hidserv-stats to dir-spec

Reported by: karsten Owned by:
Priority: Medium Milestone: Tor: 0.2.7.x-final
Component: Core Tor/Tor Version:
Severity: Keywords:
Cc: asn, amj703 Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

We should add the new hidserv-stats as specified in proposal 238 and as merged to master a few weeks ago to dir-spec.

Child Tickets

Change History (6)

comment:1 Changed 5 years ago by karsten

Status: newneeds_review

asn, amj703, please review my hidserv-stats branch.

comment:2 Changed 5 years ago by nickm

Milestone: Tor: 0.2.7.x-final

comment:3 Changed 5 years ago by asn

Looks good to me. Maybe we could also reference prop238 in the obfuscation paragraph?

comment:4 Changed 5 years ago by karsten

asn, I'm not sure about referencing the proposal. My understanding of specs is that they should be self-contained and that it shouldn't be required to read further documents to (better) understand them, in particular not closed proposal.

Of course, if there's more stuff we should include from the proposal in the specification, let's do that. Which parts do you think would be important to mention?

(This is just my opinion. If we're not sure about something we should ask Nick for his opinion.)

comment:5 in reply to:  4 Changed 5 years ago by asn

Replying to karsten:

asn, I'm not sure about referencing the proposal. My understanding of specs is that they should be self-contained and that it shouldn't be required to read further documents to (better) understand them, in particular not closed proposal.

Of course, if there's more stuff we should include from the proposal in the specification, let's do that. Which parts do you think would be important to mention?

(This is just my opinion. If we're not sure about something we should ask Nick for his opinion.)

Your rationale seems plausible. I'm fine with the patch as is. If anyone gets confused or wants more information, we can change it accordingly.

comment:6 Changed 5 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Merged; thanks!

Note: See TracTickets for help on using tickets.