Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Distribute Tor binary with Brave #12932

Closed
tildelowengrimm opened this issue Jan 30, 2018 · 6 comments
Closed

Distribute Tor binary with Brave #12932

tildelowengrimm opened this issue Jan 30, 2018 · 6 comments

Comments

@tildelowengrimm
Copy link

For a user-facing implementation of Tor tabs, we should ship Tor with Brave and run it ourselves. We might want to look to the Widevine distribution model as a template.

@riastradh-brave
Copy link
Contributor

The main cost here is a ~2.5 MB executable, judging by the Ubuntu package for tor. We could do one of two main approaches:

  1. Unconditionally bundle a tor executable with Brave.

  2. Automatically download tor as an executable on demand.

@tildelowengrimm
Copy link
Author

I definitely think we should bundle Tor with Brave. Tor will be required for the default mode of a core feature — private tabs. Having to download Tor before browsing for the first time in a private tab seems like a no-go to me. 2.5mb is a perfectly-reasonable price to pay.

@riastradh-brave
Copy link
Contributor

We have decided to bundle the tor daemon with Brave rather than download it on-demand. There are two questions about how to bundle it:

  1. Include the tor source code under third_party somewhere, and build it as part of our build by causing something to run ./configure && make && make install or by writing the necessary BUILD.gn files based on tor's makefiles (and also making sure to build and run the tests).

  2. Include the binary builds of the tor executable from the upstream Tor Project, and implement some kind of rig to verify that upstream's builds are reproducible, even if we don't necessarily build the tor executable during our regular brave build.

For option (2), we can defer the auditing temporarily, but I would be uncomfortable if we didn't eventually create the rig to verify upstream's builds. The total effort is a bit of a tossup one we figure the cost of verifying the builds into it, and just including the source code is certainly more flexible, so I would suggest option (1).

@diracdeltas
Copy link
Member

discussed in person

we found an unhelpful link https://dist.torproject.org/

@riastradh-brave to email torproject (cc'ing us) and ask what we should do

@riastradh-brave
Copy link
Contributor

I asked upstream and isis told me that the Tor Project does not distribute binary builds of the tor daemon itself outside the Tor Browser. So, we could either try to extract it from the Tor Browser distributions, or just integrate a source build into our build system. I'm more inclined to do the latter now.

@tildelowengrimm tildelowengrimm added this to the 0.24.x (Nightly Channel) milestone Apr 4, 2018
@bsclifton bsclifton modified the milestones: 0.24.x (Nightly Channel), 0.23.x (Developer Channel) May 2, 2018
@diracdeltas
Copy link
Member

done in tor/0.23.x

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

No branches or pull requests

7 participants