Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#13212 closed enhancement (implemented)

add REASON= string to SEND_HS_DESC_FAILED_EVENT()

Reported by: arma Owned by:
Priority: Medium Milestone: Tor: 0.2.6.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: tor-client
Cc: rl1987@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor: SponsorR

Description

There are 4 places that call SEND_HS_DESC_FAILED_EVENT. If we pass another argument to tell the controller which one it was, then the controller will be better prepared to inform the user of what's wrong with the hidden service she's trying to reach.

Child Tickets

Change History (4)

comment:2 Changed 5 years ago by rl1987

Cc: rl1987@… added

comment:3 Changed 5 years ago by nickm

Resolution: implemented
Status: needs_reviewclosed

looks good to me; merging! (I hope all the controllers can handle it. If not, we need to revert or make it controlled by a flag.)

comment:4 Changed 4 years ago by dgoulet

Keywords: SponsorR removed
Sponsor: SponsorR
Note: See TracTickets for help on using tickets.