Opened 3 months ago

Closed 7 weeks ago

#29357 closed enhancement (fixed)

add an ActiveOnStartup config option

Reported by: mcs Owned by: nickm
Priority: Very High Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tbb-needs, 040-proposed, 040-must
Cc: brade, gk Actual Points: .1
Parent ID: Points: .5
Reviewer: ahf Sponsor:

Description

When Tor Browser starts tor, it does not make sense to start in dormant mode (the browser is an interactive application which people will typically immediately begin using to surf the web). It would be very helpful if tor had an ActiveOnStartup Boolean config option which would ensure that tor starts in active mode.

Related: #29045

Child Tickets

TicketTypeStatusOwnerSummary
#29986defectclosedwishlist

Change History (18)

comment:1 Changed 3 months ago by nickm

Keywords: 040-proposed added
Milestone: Tor: unspecified

comment:2 Changed 3 months ago by nickm

Milestone: Tor: unspecifiedTor: 0.4.0.x-final

comment:3 Changed 3 months ago by nickm

Keywords: 040-must added

Marking tickets as 040-must based on triage with dgoulet.

comment:4 Changed 3 months ago by nickm

Owner: set to nickm
Priority: MediumVery High
Status: newaccepted

comment:5 Changed 3 months ago by nickm

Points: .5

comment:6 Changed 2 months ago by nickm

Actual Points: .1
Cc: gk added
Status: acceptedneeds_review
Type: defectenhancement

I've done an implementation in ticket29357_040, PR at https://github.com/torproject/tor/pull/800

comment:7 Changed 2 months ago by asn

Reviewer: mikeperry

comment:8 in reply to:  6 Changed 2 months ago by mcs

Replying to nickm:

I've done an implementation in ticket29357_040, PR at https://github.com/torproject/tor/pull/800

Thanks for working on this. For the record, Kathy Brade and I believe the proposed DormantCanceledByStartup option will meet our (Tor Browser's) needs.

comment:9 Changed 7 weeks ago by teor

Reviewer: mikeperry

Remove Mike as reviewer, because he's overloaded.
We'll work out what to do with these tickets in the weekly meeting.

comment:10 Changed 7 weeks ago by nickm

Reviewer: teor

comment:11 Changed 7 weeks ago by teor

Reviewer: teor

I don't need to be the reviewer on this task: someone else who is less busy can do it. Or it can just wait.

comment:12 Changed 7 weeks ago by ahf

Reviewer: ahf

comment:13 Changed 7 weeks ago by ahf

Status: needs_reviewneeds_revision

This looks good. There is a tiny typo (I think) in the changes file, once that is fixed please move this to merge_ready :-)

comment:14 Changed 7 weeks ago by nickm

Status: needs_revisionmerge_ready

Thanks! I just force-pushed a fix for the changes file.

comment:15 Changed 7 weeks ago by nickm

Keywords: asn-merge teor-merge added

comment:16 Changed 7 weeks ago by nickm

Keywords: teor-merge removed

batch-modify: asn has offered to merge these tomorrow, so removing them from teor's plate. Teor -- you can merge these anyway if you are blocked on any of them and it would save you time to do so.

comment:17 Changed 7 weeks ago by asn

Keywords: asn-merge removed

merged to 040 and forward. cannot close ticket due to child ticket being open. informing that ticket.

comment:18 Changed 7 weeks ago by asn

Resolution: fixed
Status: merge_readyclosed
Note: See TracTickets for help on using tickets.