Jigsaw-Code / Intra

An experimental tool that allows you to test new DNS-over-HTTPS services on Android
Apache License 2.0
1.54k stars 260 forks source link

Incorporate updated outline-go-tun2socks #410

Closed bemasc closed 3 years ago

bemasc commented 3 years ago

This slightly changes the names used in the Go-Java interface and adds new arguments for certificate authentication (not yet used).

bemasc commented 3 years ago

This essentially the same as #406, but I wanted to make sure we were including the latest build of outline-go-tun2socks. I was also surprised by the large drop in binary size in #406, and am similarly surprised by the different reduction in size observed here.

In a basic smoketest, the build appears to be working.

ignoramous commented 3 years ago

We're seeing ~10MB reduction in size in tun2socks.aar. Not sure what triggered that... but we were being very lax with conflicts resulting from go.mod / go.sum changes.

We have since released it to 1000 odd testers (Android 6+), it has been 2 weeks and no one has reported any issue thus far.

bemasc commented 3 years ago

Thanks for the update @ignoramous.

SeanBurford commented 3 years ago

On the size difference:

libgojni.so, which is built and included for several architectures, accounts for the size difference.

objdump -h libgojni.so (x86) shows that:

Then, comparing the symbols we see that a lot of support libs (diff --side-by-side /tmp/pre /tmp/post using objdump -t libgojni.so | awk '/.text/{print $6, $5}'| sort > /tmp/post) have been refactored out: