Opened 6 weeks ago

Closed 3 weeks ago

#28026 closed defect (fixed)

make hs-v3 client auth configuration clearer in man page

Reported by: mtigas Owned by:
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-hs, tor-doc
Cc: Actual Points:
Parent ID: Points:
Reviewer: asn Sponsor:

Description

Re: convo w/ asn yesterday on IRC (tor-dev).

Jumping off of #27547, this is my attempt at clearing up some things I thought should be made more explicit in the man page, such as not needing to set HiddenServiceAuthorizeClient, ClientOnionAuthDir doesn't contain just one file literally named ".auth_private", it's "some_handle.auth_private", etc.

This depends on #25796 (& my patch in this comment https://trac.torproject.org/projects/tor/ticket/25796#comment:10 ), since I'm following the convention in other parts of the manpage to point to spec.torproject.org, but rend-spec-v3 is not yet on that page.

Child Tickets

TicketStatusOwnerSummaryComponent
#27953closedAuthorization types for v3 onion service have to be clarified in documentationCore Tor/Tor

Attachments (1)

hs-v3-manpage.diff (3.4 KB) - added by mtigas 6 weeks ago.

Download all attachments as: .zip

Change History (9)

Changed 6 weeks ago by mtigas

Attachment: hs-v3-manpage.diff added

comment:1 Changed 6 weeks ago by mtigas

patch attached; github PR here: https://github.com/torproject/tor/pull/403

comment:2 Changed 6 weeks ago by traumschule

#27680 wants to know about this.

comment:3 Changed 6 weeks ago by traumschule

Keywords: hs-auth added

Let onion service authorization related tickets know of each other.

https://gitweb.torproject.org/torspec.git/tree/rend-spec-v3.txt#n615

[TODO: Also specify stealth client authorization.]
(NOTE: client authorization is not implemented as of 0.3.2.1-alpha.)

comment:4 Changed 5 weeks ago by nickm

Status: newneeds_review

comment:5 Changed 5 weeks ago by dgoulet

Reviewer: asn

comment:6 Changed 5 weeks ago by asn

Status: needs_reviewmerge_ready

LGTM! But we should wait until #25796 gets done to merge this.

comment:7 Changed 4 weeks ago by arma

(#25796 is now merged)

comment:8 Changed 3 weeks ago by dgoulet

Keywords: tor-hs tor-doc added; hs-auth removed
Resolution: fixed
Status: merge_readyclosed

Merged on 035 and forward. Thanks!

The 035 branch that I merged in from the PR: ticket28026_035_01. I added a changes file.

Note: See TracTickets for help on using tickets.