Opened 20 months ago

Closed 4 months ago

Last modified 4 months ago

#28763 closed defect (wontfix)

Create Tor-Android Project

Reported by: sisbell Owned by: sisbell
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-mobile, tbb-rbm, tbb-parity, TorBrowserTeam202004
Cc: sisbell, tbb-team Actual Points:
Parent ID: #28704 Points: .5
Reviewer: Sponsor:

Description

The tor-android repo contains a TorResourceInstaller. This is an Android/Java library that Orbot uses. We need this.

The tor-android repo also contains the native Tor libraries. We don't want to use these. So the Tor-Android project will need to depend on our own versions of these native libraries.

https://github.com/n8fr8/tor-android

Child Tickets

Change History (17)

comment:1 Changed 20 months ago by sisbell

The projects that tor-android is dependent on are:

  1. libevent (included patch)
  2. openssl
  3. tor
  4. zstd -https://github.com/facebook/zstd (not sure how the project is using this)
  5. lzma (not sure how the project is using this)

The first 3 we already have tor-browser-build projects for.

https://github.com/n8fr8/tor-android/tree/master/external

comment:2 in reply to:  1 Changed 20 months ago by gk

Replying to sisbell:

The projects that tor-android is dependent on are:

  1. libevent (included patch)
  2. openssl
  3. tor
  4. zstd -https://github.com/facebook/zstd (not sure how the project is using this)
  5. lzma (not sure how the project is using this)

We have the last two on our radar for desktop platforms, see #22341. But there has not been done any work to tackle that bug. As I said in that ticket starting with zstd sounds good. I am fine supporting lzma right from the beginning, too. (What about zlib support which we currently have on desktop?)

comment:3 Changed 19 months ago by gk

Keywords: TorBrowserTeam201901 added; TorBrowserTeam201812 removed

Moving tickets to Jan 2019.

comment:4 Changed 17 months ago by gk

Keywords: tbb-parity added

tbb-parity items.

comment:5 Changed 10 months ago by sysrqb

Points: .5

comment:6 Changed 10 months ago by eighthave

I highly recommend getting rid of all the asset unpacking code from Orbot. It is ancient and long unneeded, and indeed problematic. The libtor.so created by https://github.com/guardianproject/tor-android/pull/21 can be directly included in the TBB APKs by putting them in the right place on the file system before building. In tor-android, this is done in tor-android-binary/src/main/libs with tor-android-binary/src/main/jniLibs a symlink to that.

Then Android will automatically handle all the unpacking in a much more secure way. And it can be either loaded as a shared library without setting any paths, e.g. System.load("tor") or directly executed as a daemon from /data/data/org.torproject.torbrowser/lib/libtor.so.

If you still need to unpack other assets like torrc, geoip, etc., those should be put into the projects assets/ folder, then there is a standard Android method for getting those asset files. I think Orbot was already doing that.

Then TBB will need nothing from the https://github.com/guardianproject/tor-android project anymore.

comment:7 Changed 9 months ago by sysrqb

Keywords: TorBrowserTeam201911 added; TorBrowserTeam201901 removed
Owner: changed from tbb-team to sisbell
Status: newassigned

comment:8 Changed 9 months ago by sysrqb

Cc: tbb-team added

comment:9 Changed 8 months ago by pili

Keywords: TorBrowserTeam201912 added; TorBrowserTeam201911 removed

Moving tickets to December

comment:10 Changed 7 months ago by sysrqb

Keywords: TorBrowserTeam202001 added; TorBrowserTeam201912 removed

comment:11 Changed 6 months ago by pili

Keywords: TorBrowserTeam202002 added; TorBrowserTeam202001 removed

Moving tickets to February

comment:12 Changed 5 months ago by pili

Keywords: TorBrowserTeam202003 added; TorBrowserTeam202002 removed

We are no longer in February, moving tickets

comment:13 Changed 4 months ago by pili

Sponsor: Sponsor58

comment:14 Changed 4 months ago by pili

Keywords: TorBrowserTeam202004 added; TorBrowserTeam202003 removed

We are no longer in March

comment:15 Changed 4 months ago by sysrqb

Parent ID: #28704

This is not a blocker of #28704

comment:16 Changed 4 months ago by sysrqb

Parent ID: #28704
Resolution: wontfix
Status: assignedclosed

Wrong ticket. But as eighthave said, we don't need this piece, so closing.

comment:17 Changed 4 months ago by pili

Sponsor: Sponsor58
Note: See TracTickets for help on using tickets.