Opened 10 months ago

Closed 7 months ago

#28182 closed enhancement (fixed)

spec: Add to control-spec.txt some pluggable transport events

Reported by: dgoulet Owned by: dgoulet
Priority: Medium Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-spec, tor-pt, 040-roadmap-subtask, postfreeze-ok
Cc: Actual Points:
Parent ID: #28180 Points: 1
Reviewer: ahf Sponsor: Sponsor19

Description

Because of #28181 that will make possible for a PT to send back connection messages to Tor, we would like to send those to the control port to for instance Tor Browser can pick them up and inform the user.

For now, we only have the event TRANSPORT_LAUNCHED which indicate how/when the PT was launched.

As #28180 stated, most of the messages for now will be about the PT connection to the bridge such as "initiated", "unable to connect", "disconnected", etc...

Child Tickets

Change History (17)

comment:1 Changed 10 months ago by dgoulet

Keywords: 036-roadmap-subtask added

comment:2 Changed 9 months ago by nickm

Milestone: Tor: 0.3.6.x-finalTor: 0.4.0.x-final

Tor 0.3.6.x has been renamed to 0.4.0.x.

comment:3 Changed 9 months ago by teor

Keywords: 040-roadmap-subtask added; 036-roadmap-subtask removed

0.3.6 is now 0.4.0: changing keywords

comment:4 Changed 9 months ago by dgoulet

Status: newneeds_review

After implementation done in #28179, here is the updated branch:

Branch: ticket28182_02

comment:5 Changed 9 months ago by dgoulet

Reviewer: ahf

comment:6 Changed 9 months ago by dgoulet

Owner: set to dgoulet
Status: needs_reviewassigned

comment:7 Changed 9 months ago by dgoulet

Status: assignedneeds_review

comment:8 Changed 9 months ago by ahf

Status: needs_reviewmerge_ready

Looks good!

comment:9 Changed 8 months ago by nickm

Does this need to be updated with the K=V format?

comment:10 in reply to:  9 Changed 8 months ago by dgoulet

Status: merge_readyneeds_revision

Replying to nickm:

Does this need to be updated with the K=V format?

Overall yes, the STATUS message is being added as well so we need to fix the spec.

comment:11 Changed 7 months ago by dgoulet

Status: needs_revisionneeds_review

Branch: ticket28182_03

comment:12 Changed 7 months ago by ahf

Sponsor: Sponsor8Sponsor19

comment:13 Changed 7 months ago by nickm

Keywords: postfreeze-ok added

Mark some tickets as postfreeze-ok, to indicate that I think they are okay to accept in 0.4.0 post-freeze. Does not indicate that they are all necessary to do postfreeze.

comment:14 Changed 7 months ago by dgoulet

Updated branch: ticket28182_04

comment:15 Changed 7 months ago by ahf

Status: needs_reviewmerge_ready

Looks good.

comment:16 Changed 7 months ago by dgoulet

Points: 1

comment:17 Changed 7 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

Merged!

Note: See TracTickets for help on using tickets.