Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tor process reaping on Windows #1113

Closed
darkdh opened this issue Sep 13, 2018 · 1 comment · Fixed by brave/brave-core#4802
Closed

Tor process reaping on Windows #1113

darkdh opened this issue Sep 13, 2018 · 1 comment · Fixed by brave/brave-core#4802

Comments

@darkdh
Copy link
Member

darkdh commented Sep 13, 2018

Currently, tor process will be left alone when browser crash or killed on Windows.
Other platforms have different strategy to handle this situation
Mac: Terminate tor process in SuicideOnChannelErrorFilter
Linux: prctl(PR_SET_PDEATHSIG, SIGKILL) through LaunchOption.kill_on_parent_death

@bbondy bbondy added this to the 1.x Backlog milestone Sep 14, 2018
@tildelowengrimm tildelowengrimm added the priority/P4 Planned work. We expect to get to it "soon". label Oct 30, 2018
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@darkdh darkdh self-assigned this Aug 18, 2020
@darkdh darkdh added the QA/Yes label Aug 28, 2020
@darkdh darkdh added this to the 1.16.x - Nightly milestone Sep 17, 2020
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 23, 2020

Verification passed on


Brave | 1.16.67 Chromium: 86.0.4240.111 (Official Build) (64-bit)
-- | --
Revision | b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS | Windows 10 OS Version 1903 (Build 18362.1139)

  • Verified the description from the issue Tor process reaping on Windows #1113 (comment)
  • Opened a TOR window and confirmed that TOR process is shown/running in the windows Activity/processes monitoring
    image
  • Killed brave browser from process monitoring (where TOR window was already opened in brave), ensured that the TOR process is not running in Activity monitoring after killing brave browser.
  • Verified the test plan from Tor control channel brave-core#4802
OS-chosen port for tor

Initial Tor window open:

image
Confirmed one of the ports listed matches .....\AppData\Local\BraveSoftware\Brave-Browser\User Data\tor\watch\controlport

image

  • Verified that control port is different from the above step when close and reopen the tor window 2nd time

image

Tor initialization progress indicator
  • Confirmed Tor tab shows initialization progress and status:
    image

  • Confirmed Tor tab is connected once the progress status is 100 %
    image

  • Verified that tab keeps spinning when https://check.torproject.org/ is opened in an NTP as soon as TOR window is opened
    image

  • Verified the connected Tor network on https://check.torproject.org/ when tor is connected to the tor network
    image

Tor process crash handling

Killed/End the Tor process
image

Confirmed another Tor process launched 1s later.
Confirmed able to go back to the open Tor window, visit check.torproject.org and connect to Tor network.
image

Tor launcher crash handling

Killed Tor Launcher from Brave's Task Manager:

image

Went back to the open Tor tab and confirmed I was still connected to Tor network (did a hard refresh and confirmed I got a new IP address.

image
image

Closed Tor window and opened a new one. Confirmed new Process ID and able to navigate to check.torproject.org successfully:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants