July 2014 - tor-commits - lists.torproject.org

[orbot/master] make the notification larger for more display room!
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit 181b92276a1c7150e8cb4d7aaaa139b4dc2a350e Author: Nathan Freitas <nathan(a)freitas.net> Date: Thu Jul 10 11:23:34 2014 -0400 make the notification larger for more display room! --- res/layout/layout_notification_expanded.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/res/layout/layout_notification_expanded.xml b/res/layout/layout_notification_expanded.xml index e3c6dc5..c038cc8 100644 --- a/res/layout/layout_notification_expanded.xml +++ b/res/layout/layout_notification_expanded.xml @@ -18,9 +18,9 @@ xmlns:internal="http://schemas.android.com/apk/prv/res/android" android:id="@+id/status_bar_latest_event_content" android:layout_width="match_parent" - android:layout_height="64dp" - internal:layout_minHeight="64dp" - internal:layout_maxHeight="64dp" + android:layout_height="128dp" + internal:layout_minHeight="128dp" + internal:layout_maxHeight="128dp" > <ImageView android:id="@+id/icon" android:layout_width="@dimen/notification_large_icon_width"

1 0

[orbot/master] fixes for layout and notification data
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit f2f076c29c379c3288aacf403189fb945b0463b7 Author: Nathan Freitas <nathan(a)freitas.net> Date: Thu Jul 10 12:34:31 2014 -0400 fixes for layout and notification data be sure to only get the IP of the last hop (that is the exit) --- res/layout/layout_notification_expanded.xml | 63 ++++++----- src/org/torproject/android/service/TorService.java | 117 ++++++++------------ 2 files changed, 79 insertions(+), 101 deletions(-) diff --git a/res/layout/layout_notification_expanded.xml b/res/layout/layout_notification_expanded.xml index c038cc8..b61d6fe 100644 --- a/res/layout/layout_notification_expanded.xml +++ b/res/layout/layout_notification_expanded.xml @@ -18,9 +18,7 @@ xmlns:internal="http://schemas.android.com/apk/prv/res/android" android:id="@+id/status_bar_latest_event_content" android:layout_width="match_parent" - android:layout_height="128dp" - internal:layout_minHeight="128dp" - internal:layout_maxHeight="128dp" + android:layout_height="wrap_content" > <ImageView android:id="@+id/icon" android:layout_width="@dimen/notification_large_icon_width" @@ -54,7 +52,9 @@ android:ellipsize="marquee" android:fadingEdge="horizontal" android:layout_weight="1" - /> + android:textStyle="bold" + android:textAppearance="@style/NotificationTitle" + /> <ViewStub android:id="@+id/time" android:layout_width="wrap_content" android:layout_height="wrap_content" @@ -70,32 +70,12 @@ android:layout="@layout/notification_template_part_chronometer" /> </LinearLayout> - <TextView android:id="@+id/text2" - android:layout_width="match_parent" - android:layout_height="wrap_content" - android:layout_marginTop="-2dp" - android:layout_marginBottom="-2dp" - android:layout_marginStart="8dp" - android:singleLine="true" - android:fadingEdge="horizontal" - android:ellipsize="marquee" - android:visibility="gone" - /> - <ProgressBar - android:id="@android:id/progress" - android:layout_width="match_parent" - android:layout_height="12dp" - android:layout_marginStart="8dp" - android:visibility="gone" - style="?android:attr/progressBarStyleHorizontal" - /> <LinearLayout android:id="@+id/line3" android:layout_width="match_parent" android:layout_height="wrap_content" - android:orientation="horizontal" - android:gravity="center_vertical" - android:layout_marginStart="8dp" + android:orientation="horizontal" + android:layout_marginStart="8dp" > <TextView android:id="@+id/text" android:layout_width="0dp" @@ -105,6 +85,7 @@ android:singleLine="true" android:ellipsize="marquee" android:fadingEdge="horizontal" + android:textAppearance="@style/NotificationText" /> <TextView android:id="@+id/info" android:layout_width="wrap_content" @@ -114,6 +95,7 @@ android:singleLine="true" android:gravity="center" android:paddingStart="8dp" + android:textAppearance="@style/NotificationText" /> <ImageView android:id="@+id/right_icon" android:layout_width="16dp" @@ -124,13 +106,30 @@ android:scaleType="centerInside" android:visibility="gone" /> - <Button +  </LinearLayout> + + <TextView android:id="@+id/text2" + android:layout_width="match_parent" + android:layout_height="wrap_content" + android:layout_marginTop="-2dp" + android:layout_marginBottom="-2dp" + android:layout_marginStart="8dp" + android:fadingEdge="horizontal" + /> + <ProgressBar + android:id="@android:id/progress" + android:layout_width="match_parent" + android:layout_height="12dp" + android:layout_marginStart="8dp" + android:visibility="gone" + style="?android:attr/progressBarStyleHorizontal" + /> </LinearLayout> </FrameLayout> diff --git a/src/org/torproject/android/service/TorService.java b/src/org/torproject/android/service/TorService.java index 9ed28b9..baaa0a2 100644 --- a/src/org/torproject/android/service/TorService.java +++ b/src/org/torproject/android/service/TorService.java @@ -23,15 +23,14 @@ import java.util.Arrays; import java.util.Iterator; import java.util.List; import java.util.StringTokenizer; -import java.util.Timer; -import java.util.TimerTask; +import java.util.concurrent.TimeoutException; import java.util.regex.Matcher; import java.util.regex.Pattern; -import java.util.concurrent.TimeoutException; import net.freehaven.tor.control.ConfigEntry; import net.freehaven.tor.control.EventHandler; import net.freehaven.tor.control.TorControlConnection; +import net.freehaven.tor.control.TorControlError; import org.sufficientlysecure.rootcommands.Shell; import org.sufficientlysecure.rootcommands.command.SimpleCommand; @@ -58,16 +57,13 @@ import android.net.ConnectivityManager; import android.net.NetworkInfo; import android.os.AsyncTask; import android.os.Environment; -import android.os.Handler; import android.os.IBinder; import android.os.RemoteCallbackList; import android.os.RemoteException; import android.support.v4.app.NotificationCompat; import android.support.v4.app.NotificationCompat.Builder; -import android.support.v4.content.LocalBroadcastManager; import android.util.Log; import android.widget.RemoteViews; -import android.widget.Toast; public class TorService extends Service implements TorServiceConstants, TorConstants, EventHandler { @@ -219,7 +215,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } @SuppressLint("NewApi") - private void showToolbarNotification (String notifyMsg, int notifyId, int icon, boolean isOngoing) + private void showToolbarNotification (String notifyMsg, int notifyType, int icon) { //Reusable code. @@ -230,10 +226,14 @@ public class TorService extends Service implements TorServiceConstants, TorConst RemoteViews expandedView = new RemoteViews(this.getPackageName(), R.layout.layout_notification_expanded); - expandedView.setTextViewText(R.id.text, notifyMsg); - expandedView.setTextViewText(R.id.title, getString(R.string.app_name)+ ' ' + exitIP); - //expandedView.setTextViewText(R.id.exitIP, exitIP); - //expandedView.setOnClickPendingIntent(R.id._tor_notificationBT, pendIntent); + if (notifyType == NOTIFY_ID) + expandedView.setTextViewText(R.id.text, notifyMsg); + else + expandedView.setTextViewText(R.id.text2, notifyMsg); + + expandedView.setTextViewText(R.id.title, getString(R.string.app_name)); + expandedView.setTextViewText(R.id.info, "Tor IP: " + exitIP); + // expandedView.setOnClickPendingIntent(R.id._tor_notificationBT, pendIntent); expandedView.setImageViewResource(R.id.icon, icon); if (mNotifyBuilder == null) @@ -245,7 +245,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst { mNotifyBuilder = new NotificationCompat.Builder(this) .setContentTitle(getString(R.string.app_name)) - .setContentText( getString(R.string.status_activated)) .setSmallIcon(R.drawable.ic_stat_tor); mNotifyBuilder.setContentIntent(pendIntent); @@ -256,31 +255,24 @@ public class TorService extends Service implements TorServiceConstants, TorConst mNotifyBuilder.setContentText(notifyMsg); mNotifyBuilder.setSmallIcon(icon); - - if (notifyId == ERROR_NOTIFY_ID) + if (notifyType != NOTIFY_ID) { mNotifyBuilder.setTicker(notifyMsg); mNotifyBuilder.setLights(Color.RED, 1000, 1000); mNotifyBuilder.setSmallIcon(R.drawable.ic_stat_notifyerr); } - else - { - mNotifyBuilder.setTicker(null); //make sure to clear ticker - } + + mNotifyBuilder.setOngoing(prefPersistNotifications); mNotification = mNotifyBuilder.build(); mNotification.bigContentView = expandedView; - if (isOngoing) + if (mNotification == null && prefPersistNotifications) { - startForeground(notifyId, mNotification); - + startForeground(NOTIFY_ID, mNotification); } - else - { - mNotificationManager.notify(notifyId, mNotification); - } - + + mNotificationManager.notify(NOTIFY_ID, mNotification); } @@ -435,13 +427,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst } catch (FileNotFoundException e) { logException("unable to read onion hostname file",e); - showToolbarNotification(getString(R.string.unable_to_read_hidden_service_name), HS_NOTIFY_ID, R.drawable.ic_stat_notifyerr, false); + showToolbarNotification(getString(R.string.unable_to_read_hidden_service_name), HS_NOTIFY_ID, R.drawable.ic_stat_notifyerr); return null; } } else { - showToolbarNotification(getString(R.string.unable_to_read_hidden_service_name), HS_NOTIFY_ID, R.drawable.ic_stat_notifyerr, false); + showToolbarNotification(getString(R.string.unable_to_read_hidden_service_name), HS_NOTIFY_ID, R.drawable.ic_stat_notifyerr); return null; } @@ -451,7 +443,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst { String onionHostname = result.toString(); - showToolbarNotification(getString(R.string.hidden_service_on) + ' ' + onionHostname, HS_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.hidden_service_on) + ' ' + onionHostname, HS_NOTIFY_ID, R.drawable.ic_stat_tor); Editor pEdit = prefs.edit(); pEdit.putString("pref_hs_hostname",onionHostname); pEdit.commit(); @@ -719,13 +711,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst if(proxyAll) { - showToolbarNotification(getString(R.string.setting_up_full_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.setting_up_full_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); code = mTransProxy.setTransparentProxyingAll(this); } else { - showToolbarNotification(getString(R.string.setting_up_app_based_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.setting_up_app_based_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); code = mTransProxy.setTransparentProxyingByApp(this,AppManager.getApps(this, TorServiceUtils.getSharedPrefs(getApplicationContext()))); } @@ -735,11 +727,11 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (code == 0) { - showToolbarNotification(getString(R.string.transparent_proxying_enabled), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.transparent_proxying_enabled), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); if (enableTether) { - showToolbarNotification(getString(R.string.transproxy_enabled_for_tethering_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.transproxy_enabled_for_tethering_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); mTransProxy.enableTetheringRules(this); @@ -747,7 +739,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } else { - showToolbarNotification(getString(R.string.warning_error_starting_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.warning_error_starting_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); } @@ -1168,7 +1160,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst stopTor(); logException("Unable to start Tor: " + e.toString(),e); currentStatus = STATUS_OFF; - showToolbarNotification(getString(R.string.unable_to_start_tor) + ": " + e.getMessage(), ERROR_NOTIFY_ID, R.drawable.ic_stat_notifyerr, false); + showToolbarNotification(getString(R.string.unable_to_start_tor) + ": " + e.getMessage(), ERROR_NOTIFY_ID, R.drawable.ic_stat_notifyerr); } } else @@ -1191,7 +1183,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst { currentStatus = STATUS_ON; - showToolbarNotification(getString(R.string.status_activated), NOTIFY_ID, R.drawable.ic_stat_tor, prefPersistNotifications); + showToolbarNotification(getString(R.string.status_activated), NOTIFY_ID, R.drawable.ic_stat_tor); } @@ -1261,7 +1253,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst iconId = R.drawable.ic_stat_tor_xfer; if (mConnectivity && prefPersistNotifications) - showToolbarNotification(sb.toString(), NOTIFY_ID, iconId, prefPersistNotifications); + showToolbarNotification(sb.toString(), NOTIFY_ID, iconId); mTotalTrafficWritten += written; mTotalTrafficRead += read; @@ -1321,8 +1313,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst public void circuitStatus(String status, String circID, String path) { - if (status.equals("BUILT") || status.equals("CLOSED")) - { + StringBuilder sb = new StringBuilder(); sb.append("Circuit ("); sb.append((circID)); @@ -1332,41 +1323,25 @@ public class TorService extends Service implements TorServiceConstants, TorConst StringTokenizer st = new StringTokenizer(path,","); String node = null; + String nodeName = null; while (st.hasMoreTokens()) { node = st.nextToken(); - final String nodeName = parseNodeName(node); + nodeName = parseNodeName(node); sb.append(nodeName); - if(status.equals("BUILT") && currentStatus==STATUS_ON){ - new getExternalIP().execute(nodeName); - } - if (st.hasMoreTokens()) sb.append (" > "); } logNotice(sb.toString()); - } - - /** - if (currentStatus != STATUS_ON) - { - try { - String state; - state = conn.getInfo("dormant"); - - if (state != null && Integer.parseInt(state) == 0) - currentStatus = STATUS_ON; - else - currentStatus = STATUS_CONNECTING; - - } catch (IOException e) { - logException("Error getting state from Tor control port",e); + //get IP from last nodename + if(status.equals("BUILT") && currentStatus==STATUS_ON){ + new getExternalIP().execute(nodeName); } - }*/ + } @@ -1386,7 +1361,8 @@ public class TorService extends Service implements TorServiceConstants, TorConst } return nodeDetails; - + } catch (TorControlError e) { + Log.d(TorService.TAG,"Error getting node details for: " + params[0]); } catch (Exception e) { Log.e(TorService.TAG,"Error getting node details",e); } @@ -1402,6 +1378,9 @@ public class TorService extends Service implements TorServiceConstants, TorConst Pattern pattern = Pattern.compile(IPADDRESS_PATTERN); Matcher matcher = null; if(result!=null){ + + sendCallbackLogMessage("Node Info: " + result); + matcher = pattern.matcher(result); if (matcher.find()) { exitIP = matcher.group(); @@ -1837,13 +1816,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (!mConnectivity) { logNotice(context.getString(R.string.no_network_connectivity_putting_tor_to_sleep_)); - showToolbarNotification(getString(R.string.no_internet_connection_tor),NOTIFY_ID,R.drawable.ic_stat_tor_off,prefPersistNotifications); + showToolbarNotification(getString(R.string.no_internet_connection_tor),NOTIFY_ID,R.drawable.ic_stat_tor_off); } else { logNotice(context.getString(R.string.network_connectivity_is_good_waking_tor_up_)); - showToolbarNotification(getString(R.string.status_activated),NOTIFY_ID,R.drawable.ic_stat_tor,prefPersistNotifications); + showToolbarNotification(getString(R.string.status_activated),NOTIFY_ID,R.drawable.ic_stat_tor); if (mHasRoot && mEnableTransparentProxy) enableTransparentProxy(mTransProxyAll, mTransProxyTethering); @@ -1940,7 +1919,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } catch (Exception e) { - showToolbarNotification (getString(R.string.error_installing_binares),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr, false); + showToolbarNotification (getString(R.string.error_installing_binares),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr); return false; } @@ -1963,7 +1942,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst { String msgBridge = getString(R.string.bridge_requires_ip) + getString(R.string.send_email_for_bridges); - showToolbarNotification(msgBridge, ERROR_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(msgBridge, ERROR_NOTIFY_ID, R.drawable.ic_stat_tor); logMessage(msgBridge); return false; @@ -1977,7 +1956,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst bridgeDelim = ","; } - showToolbarNotification(getString(R.string.notification_using_bridges) + ": " + bridgeList, TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor, false); + showToolbarNotification(getString(R.string.notification_using_bridges) + ": " + bridgeList, TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); StringTokenizer st = new StringTokenizer(bridgeList,bridgeDelim); while (st.hasMoreTokens()) @@ -2036,7 +2015,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } catch (Exception e) { - showToolbarNotification (getString(R.string.your_reachableaddresses_settings_caused_an_exception_),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr, false); + showToolbarNotification (getString(R.string.your_reachableaddresses_settings_caused_an_exception_),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr); return false; } @@ -2065,7 +2044,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } catch (Exception e) { - showToolbarNotification (getString(R.string.your_relay_settings_caused_an_exception_),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr, false); + showToolbarNotification (getString(R.string.your_relay_settings_caused_an_exception_),ERROR_NOTIFY_ID,R.drawable.ic_stat_notifyerr); return false;

1 0

[orbot/master] small cleanup of asynctask and error handling
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit a6da4bf5306d794b50aeb2b043485d151537841e Author: Nathan Freitas <nathan(a)freitas.net> Date: Thu Jul 10 11:23:49 2014 -0400 small cleanup of asynctask and error handling --- src/org/torproject/android/service/TorService.java | 22 ++++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/org/torproject/android/service/TorService.java b/src/org/torproject/android/service/TorService.java index 4d5bb6d..9ed28b9 100644 --- a/src/org/torproject/android/service/TorService.java +++ b/src/org/torproject/android/service/TorService.java @@ -231,7 +231,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst R.layout.layout_notification_expanded); expandedView.setTextViewText(R.id.text, notifyMsg); - expandedView.setTextViewText(R.id.title, "ORBOT "+exitIP); + expandedView.setTextViewText(R.id.title, getString(R.string.app_name)+ ' ' + exitIP); //expandedView.setTextViewText(R.id.exitIP, exitIP); //expandedView.setOnClickPendingIntent(R.id._tor_notificationBT, pendIntent); expandedView.setImageViewResource(R.id.icon, icon); @@ -1370,42 +1370,42 @@ public class TorService extends Service implements TorServiceConstants, TorConst } - private class getExternalIP extends AsyncTask<String, Void, Void>{ + private class getExternalIP extends AsyncTask<String, Void, String>{ private long time; - private String nodeDetails; @Override - protected Void doInBackground(String... params) { + protected String doInBackground(String... params) { time = System.nanoTime(); try { - nodeDetails = conn.getInfo("ns/name/"+params[0]); + String nodeDetails = conn.getInfo("ns/name/"+params[0]); if (ENABLE_DEBUG_LOG) { Log.d(TAG,"Node Details: "+nodeDetails); sendCallbackLogMessage("Node Details: "+nodeDetails); } + return nodeDetails; + } catch (Exception e) { - // TODO Auto-generated catch block - e.printStackTrace(); + Log.e(TorService.TAG,"Error getting node details",e); } return null; } @Override - protected void onPostExecute(Void result) { + protected void onPostExecute(String result) { // check if we need to update the exit IP if(time > exitIPTime) { exitIPTime = time; Pattern pattern = Pattern.compile(IPADDRESS_PATTERN); Matcher matcher = null; - if(nodeDetails!=null){ - matcher = pattern.matcher(nodeDetails); + if(result!=null){ + matcher = pattern.matcher(result); if (matcher.find()) { - Log.d(TAG, "ip: "+matcher.group()); exitIP = matcher.group(); + sendCallbackLogMessage("Exit IP: "+exitIP); } } }

1 0

[orbot/master] update commit pegs for external
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit c7409b78c9ecfa4d319ba559e4843d426ab2a62a Author: Nathan Freitas <nathan(a)freitas.net> Date: Thu Jul 10 12:34:53 2014 -0400 update commit pegs for external --- external/libevent | 2 +- external/openssl | 2 +- external/polipo | 2 +- external/tor | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/external/libevent b/external/libevent index ce46db9..6417777 160000 --- a/external/libevent +++ b/external/libevent @@ -1 +1 @@ -Subproject commit ce46db99b94389a8fed0248f64f88d047cdac416 +Subproject commit 64177777165d9684bafbfa946abd126f7ebff11f diff --git a/external/openssl b/external/openssl index b2d951e..6b72417 160000 --- a/external/openssl +++ b/external/openssl @@ -1 +1 @@ -Subproject commit b2d951e4232d2f90168f9a3dd0b7df9ecf2d81a8 +Subproject commit 6b72417a00b803680edc9be214c1b3dd9dae203d diff --git a/external/polipo b/external/polipo index af08a10..f690e76 160000 --- a/external/polipo +++ b/external/polipo @@ -1 +1 @@ -Subproject commit af08a108acb9ac717f114d81c60cad0ccc0d7c21 +Subproject commit f690e763257ba05513b0bf0bd8a99554a0719201 diff --git a/external/tor b/external/tor index d2147cc..2ee56e4 160000 --- a/external/tor +++ b/external/tor @@ -1 +1 @@ -Subproject commit d2147cc7ba5c5d5114bae61000cbf757353c3ed6 +Subproject commit 2ee56e4c2c841a45418cfb826e1ce6689278382d

1 0

[orbot/master] re-arch status callbacks and improve notification
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit e1ec776cb843cf5fdf5a37c299614af7dcf448df Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 09:19:30 2014 -0400 re-arch status callbacks and improve notification --- res/layout/layout_notification_expanded.xml | 10 +- res/raw/torrc | 3 +- res/raw/torrcdiag | 7 - src/org/torproject/android/Orbot.java | 136 ++--- .../android/OrbotDiagnosticsActivity.java | 2 +- .../torproject/android/service/ITorService.aidl | 16 +- .../android/service/ITorServiceCallback.aidl | 24 - .../android/service/TorResourceInstaller.java | 7 +- src/org/torproject/android/service/TorService.java | 536 +++++++++----------- .../torproject/android/service/TorTransProxy.java | 2 +- 10 files changed, 345 insertions(+), 398 deletions(-) diff --git a/res/layout/layout_notification_expanded.xml b/res/layout/layout_notification_expanded.xml index b61d6fe..c208583 100644 --- a/res/layout/layout_notification_expanded.xml +++ b/res/layout/layout_notification_expanded.xml @@ -114,7 +114,15 @@ android:src="@drawable/nav_refresh" /> --> </LinearLayout> - + <LinearLayout + android:layout_width="match_parent" + android:layout_height="1dp" + android:orientation="horizontal" + android:background="#999999" + android:layout_marginTop="3dp" + android:layout_marginBottom="3dp" + + /> <TextView android:id="@+id/text2" android:layout_width="match_parent" android:layout_height="wrap_content" diff --git a/res/raw/torrc b/res/raw/torrc index 2a5a48a..1661704 100644 --- a/res/raw/torrc +++ b/res/raw/torrc @@ -3,5 +3,4 @@ ControlPort auto SOCKSPort 0 DNSPort 0 TransPort 0 -CookieAuthentication 1 -AvoidDiskWrites 1 \ No newline at end of file +CookieAuthentication 1 \ No newline at end of file diff --git a/res/raw/torrcdiag b/res/raw/torrcdiag deleted file mode 100644 index b8f9661..0000000 --- a/res/raw/torrcdiag +++ /dev/null @@ -1,7 +0,0 @@ -Log debug stdout -ControlPort auto -SOCKSPort 0 -DNSPort 0 -TransPort 0 -CookieAuthentication 1 -AvoidDiskWrites 1 \ No newline at end of file diff --git a/src/org/torproject/android/Orbot.java b/src/org/torproject/android/Orbot.java index 35d508e..06ffe57 100644 --- a/src/org/torproject/android/Orbot.java +++ b/src/org/torproject/android/Orbot.java @@ -9,7 +9,6 @@ import java.net.URLDecoder; import java.util.Locale; import org.torproject.android.service.ITorService; -import org.torproject.android.service.ITorServiceCallback; import org.torproject.android.service.TorService; import org.torproject.android.service.TorServiceConstants; import org.torproject.android.service.TorServiceUtils; @@ -774,6 +773,8 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic { try { + torStatus = mService.getStatus(); + if (torStatus != TorServiceConstants.STATUS_ON) mService.processSettings(); @@ -937,7 +938,7 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic } - else if (torStatus != newTorStatus) + else if (newTorStatus == TorServiceConstants.STATUS_OFF) { // mViewMain.setBackgroundResource(R.drawable.onionrootonlygrey); imgStatus.setImageResource(R.drawable.toroff); @@ -982,6 +983,7 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic Message msg = mHandler.obtainMessage(TorServiceConstants.ENABLE_TOR_MSG); msg.getData().putString(HANDLER_TOR_MSG, getString(R.string.status_starting_up)); mHandler.sendMessage(msg); + } else { @@ -1000,8 +1002,9 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic mService.setProfile(TorServiceConstants.PROFILE_OFF); Message msg = mHandler.obtainMessage(TorServiceConstants.DISABLE_TOR_MSG); mHandler.sendMessage(msg); + + updateStatus(""); - } @@ -1043,61 +1046,72 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic return false; } - - /** - * This implementation is used to receive callbacks from the remote - * service. - * - * If we have this setup probably, we shouldn't have to poll or query status - * to the service, as it should send it as it changes or when we bind/unbind to it - * from this activity - */ - private ITorServiceCallback mCallback = new ITorServiceCallback.Stub() { - /** - * This is called by the remote service regularly to tell us about - * new values. Note that IPC calls are dispatched through a thread - * pool running in each process, so the code executing here will - * NOT be running in our main thread like most other things -- so, - * to update the UI, we need to use a Handler to hop over there. - */ - - //receive a new string vaule end-user displayable message from the ITorService - public void statusChanged(String value) { - - //pass it off to the progressDialog - Message msg = mHandler.obtainMessage(TorServiceConstants.STATUS_MSG); - msg.getData().putString(HANDLER_TOR_MSG, value); - mHandler.sendMessage(msg); + Thread threadUpdater = null; + boolean mKeepUpdating = false; + + public void initUpdates () + { + mKeepUpdating = true; + + if (threadUpdater == null || !threadUpdater.isAlive()) + { + threadUpdater = new Thread(new Runnable() + { + + public void run () + { + + while (mKeepUpdating) + { + try + { + if (mService != null) + { + for (String log : mService.getLog()) + { + Message msg = mHandler.obtainMessage(TorServiceConstants.LOG_MSG); + msg.getData().putString(HANDLER_TOR_MSG, log); + mHandler.sendMessage(msg); + } + + for (String status : mService.getStatusMessage()) + { + Message msg = mHandler.obtainMessage(TorServiceConstants.STATUS_MSG); + msg.getData().putString(HANDLER_TOR_MSG, status); + mHandler.sendMessage(msg); + } + + long[] bws = mService.getBandwidth(); + Message msg = mHandler.obtainMessage(TorServiceConstants.MESSAGE_TRAFFIC_COUNT); + msg.getData().putLong("download", bws[0]); + msg.getData().putLong("upload", bws[1]); + msg.getData().putLong("readTotal", bws[2]); + msg.getData().putLong("writeTotal", bws[3]); + mHandler.sendMessage(msg); + + try { Thread.sleep(1000); } + catch (Exception e){} + + + torStatus = mService.getStatus(); + } + } + catch (RemoteException re) + { + Log.e(TAG, "error getting service updates",re); + } + } + + } + }); + + threadUpdater.start(); + + } } - - @Override - public void logMessage(String value) throws RemoteException { - - Message msg = mHandler.obtainMessage(TorServiceConstants.LOG_MSG); - msg.getData().putString(HANDLER_TOR_MSG, value); - mHandler.sendMessage(msg); - - } - - @Override - public void updateBandwidth(long upload, long download, long written, long read) { - - Message msg = Message.obtain(); - msg.what = TorServiceConstants.MESSAGE_TRAFFIC_COUNT; - - Bundle data = new Bundle(); - data.putLong("upload", upload); - data.putLong("download", download); - data.putLong("readTotal",read); - data.putLong("writeTotal",written); - - msg.setData(data); - mHandler.sendMessage(msg); - - } - }; - + + // this is what takes messages or values from the callback threads or other non-mainUI threads //and passes them back into the main UI thread for display to the user @@ -1177,13 +1191,13 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic // representation of that from the raw service object. mService = ITorService.Stub.asInterface(service); - torStatus = TorServiceConstants.STATUS_OFF; // We want to monitor the service for as long as we are // connected to it. try { - mService.registerCallback(mCallback); - + torStatus = mService.getStatus(); + initUpdates(); + if (autoStartFromIntent) { @@ -1205,15 +1219,15 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic } - - } public void onServiceDisconnected(ComponentName className) { + // This is called when the connection with the service has been // unexpectedly disconnected -- that is, its process crashed. + mKeepUpdating = false; mService = null; Log.d(TAG,"service was disconnected"); } diff --git a/src/org/torproject/android/OrbotDiagnosticsActivity.java b/src/org/torproject/android/OrbotDiagnosticsActivity.java index 717d3ee..4cc8c6b 100644 --- a/src/org/torproject/android/OrbotDiagnosticsActivity.java +++ b/src/org/torproject/android/OrbotDiagnosticsActivity.java @@ -119,7 +119,7 @@ public class OrbotDiagnosticsActivity extends Activity { File fileTor= new File(appBinHome, TorServiceConstants.TOR_ASSET_KEY); enableBinExec (fileTor, appBinHome); - InputStream is = getResources().openRawResource(R.raw.torrcdiag); + InputStream is = getResources().openRawResource(R.raw.torrc); File fileTorrc = new File(appBinHome, TorServiceConstants.TORRC_ASSET_KEY + "diag"); TorResourceInstaller.streamToFile(is,fileTorrc, false, false); diff --git a/src/org/torproject/android/service/ITorService.aidl b/src/org/torproject/android/service/ITorService.aidl index 4891b55..2d57c9b 100644 --- a/src/org/torproject/android/service/ITorService.aidl +++ b/src/org/torproject/android/service/ITorService.aidl @@ -1,6 +1,5 @@ package org.torproject.android.service; -import org.torproject.android.service.ITorServiceCallback; /** * an interface for calling on to a remote service @@ -8,16 +7,6 @@ import org.torproject.android.service.ITorServiceCallback; interface ITorService { /** - * This allows Tor service to send messages back to the GUI - */ - void registerCallback(ITorServiceCallback cb); - - /** - * Remove registered callback interface. - */ - void unregisterCallback(ITorServiceCallback cb); - - /** * Get a simple int status value for the state of Tor **/ int getStatus(); @@ -58,4 +47,9 @@ interface ITorService { */ void newIdentity (); + String[] getStatusMessage (); + + String[] getLog (); + + long[] getBandwidth (); } diff --git a/src/org/torproject/android/service/ITorServiceCallback.aidl b/src/org/torproject/android/service/ITorServiceCallback.aidl deleted file mode 100644 index b7b6827..0000000 --- a/src/org/torproject/android/service/ITorServiceCallback.aidl +++ /dev/null @@ -1,24 +0,0 @@ -package org.torproject.android.service; - -/** - * Callback interface used to send - * synchronous notifications back to its clients. Note that this is a - * one-way interface so the server does not block waiting for the client. - */ -oneway interface ITorServiceCallback { - /** - * Called when the service has a something to display to the user - */ - void statusChanged(String value); - - /** - * Called when the service returns the bandwidth user to display to the user - */ - void updateBandwidth(long upload, long download, long written, long read); - - /** - * Called when the service has something to add to the log - */ - void logMessage(String value); - -} diff --git a/src/org/torproject/android/service/TorResourceInstaller.java b/src/org/torproject/android/service/TorResourceInstaller.java index 3e91ea4..923e841 100644 --- a/src/org/torproject/android/service/TorResourceInstaller.java +++ b/src/org/torproject/android/service/TorResourceInstaller.java @@ -77,12 +77,7 @@ public class TorResourceInstaller implements TorServiceConstants { outFile = new File(installFolder, TORRC_ASSET_KEY); shell.add(new SimpleCommand(COMMAND_RM_FORCE + outFile.getAbsolutePath())).waitForFinish(); streamToFile(is,outFile, false, false); - - is = context.getResources().openRawResource(R.raw.torrcdiag); - outFile = new File(installFolder, TORRCDIAG_ASSET_KEY); - shell.add(new SimpleCommand(COMMAND_RM_FORCE + outFile.getAbsolutePath())).waitForFinish(); - streamToFile(is,outFile, false, false); - + is = context.getResources().openRawResource(R.raw.torrctether); outFile = new File(installFolder, TORRC_TETHER_KEY); shell.add(new SimpleCommand(COMMAND_RM_FORCE + outFile.getAbsolutePath())).waitForFinish(); diff --git a/src/org/torproject/android/service/TorService.java b/src/org/torproject/android/service/TorService.java index baaa0a2..f72ba31 100644 --- a/src/org/torproject/android/service/TorService.java +++ b/src/org/torproject/android/service/TorService.java @@ -9,6 +9,7 @@ package org.torproject.android.service; import java.io.BufferedReader; +import java.io.ByteArrayOutputStream; import java.io.DataInputStream; import java.io.File; import java.io.FileInputStream; @@ -16,15 +17,22 @@ import java.io.FileNotFoundException; import java.io.FileReader; import java.io.FileWriter; import java.io.IOException; +import java.io.InputStreamReader; +import java.io.PrintStream; import java.io.PrintWriter; +import java.net.InetSocketAddress; +import java.net.Proxy; import java.net.Socket; +import java.net.URL; +import java.net.URLConnection; import java.util.ArrayList; import java.util.Arrays; +import java.util.HashMap; import java.util.Iterator; import java.util.List; +import java.util.Set; import java.util.StringTokenizer; import java.util.concurrent.TimeoutException; -import java.util.regex.Matcher; import java.util.regex.Pattern; import net.freehaven.tor.control.ConfigEntry; @@ -32,6 +40,8 @@ import net.freehaven.tor.control.EventHandler; import net.freehaven.tor.control.TorControlConnection; import net.freehaven.tor.control.TorControlError; +import org.json.JSONArray; +import org.json.JSONObject; import org.sufficientlysecure.rootcommands.Shell; import org.sufficientlysecure.rootcommands.command.SimpleCommand; import org.torproject.android.Orbot; @@ -56,9 +66,8 @@ import android.graphics.Color; import android.net.ConnectivityManager; import android.net.NetworkInfo; import android.os.AsyncTask; -import android.os.Environment; +import android.os.Build; import android.os.IBinder; -import android.os.RemoteCallbackList; import android.os.RemoteException; import android.support.v4.app.NotificationCompat; import android.support.v4.app.NotificationCompat.Builder; @@ -84,9 +93,9 @@ public class TorService extends Service implements TorServiceConstants, TorConst private static final int HS_NOTIFY_ID = 4; private boolean prefPersistNotifications = true; - private String IPADDRESS_PATTERN = + private static final String IPADDRESS_PATTERN = "(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)"; - private long exitIPTime = 0; + private final static Pattern pattern = Pattern.compile(IPADDRESS_PATTERN); private static final int MAX_START_TRIES = 3; @@ -117,20 +126,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst private NotificationManager mNotificationManager = null; private Builder mNotifyBuilder; private Notification mNotification; - private String exitIP = ""; private boolean mHasRoot = false; private boolean mEnableTransparentProxy = false; private boolean mTransProxyAll = false; private boolean mTransProxyTethering = false; - - private ArrayList<String> callbackBuffer = new ArrayList<String>(); - private boolean inCallbackStatus = false; - private boolean inCallback = false; - - - public void logMessage(String msg) + public void debug(String msg) { if (ENABLE_DEBUG_LOG) { @@ -145,9 +147,15 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (ENABLE_DEBUG_LOG) { Log.e(TAG,msg,e); + ByteArrayOutputStream baos = new ByteArrayOutputStream(); + e.printStackTrace(new PrintStream(baos)); + + sendCallbackLogMessage(msg + '\n'+ new String(baos.toByteArray())); + } - - sendCallbackLogMessage(msg); + else + sendCallbackLogMessage(msg); + } @@ -212,9 +220,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (mNotificationManager != null) mNotificationManager.cancelAll(); + + hmBuiltNodes.clear(); + + } - @SuppressLint("NewApi") + @SuppressLint("NewApi") private void showToolbarNotification (String notifyMsg, int notifyType, int icon) { @@ -226,13 +238,44 @@ public class TorService extends Service implements TorServiceConstants, TorConst RemoteViews expandedView = new RemoteViews(this.getPackageName(), R.layout.layout_notification_expanded); + StringBuffer sbInfo = new StringBuffer(); + + if (notifyType == NOTIFY_ID) expandedView.setTextViewText(R.id.text, notifyMsg); else - expandedView.setTextViewText(R.id.text2, notifyMsg); + { + expandedView.setTextViewText(R.id.info, notifyMsg); + + } + + if (hmBuiltNodes.size() > 0) + { + sbInfo.append("Your Tor Public IPs:\n"); + Set<String> itBuiltNodes = hmBuiltNodes.keySet(); + for (String key : itBuiltNodes) + { + Node node = hmBuiltNodes.get(key); + + if (node.ipAddress != null) + sbInfo.append(node.ipAddress); + + if (node.country != null) + sbInfo.append(' ').append(node.country); + + if (node.organization != null) + sbInfo.append(" (").append(node.organization).append(')'); + + sbInfo.append('\n'); + + } + + expandedView.setTextViewText(R.id.text2, sbInfo.toString()); + } + + expandedView.setTextViewText(R.id.title, getString(R.string.app_name)); - expandedView.setTextViewText(R.id.title, getString(R.string.app_name)); - expandedView.setTextViewText(R.id.info, "Tor IP: " + exitIP); + // expandedView.setTextViewText(R.id.info, infoMessage.toString()); // expandedView.setOnClickPendingIntent(R.id._tor_notificationBT, pendIntent); expandedView.setImageViewResource(R.id.icon, icon); @@ -258,15 +301,17 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (notifyType != NOTIFY_ID) { mNotifyBuilder.setTicker(notifyMsg); - mNotifyBuilder.setLights(Color.RED, 1000, 1000); - mNotifyBuilder.setSmallIcon(R.drawable.ic_stat_notifyerr); + mNotifyBuilder.setLights(Color.GREEN, 1000, 1000); } mNotifyBuilder.setOngoing(prefPersistNotifications); mNotification = mNotifyBuilder.build(); - mNotification.bigContentView = expandedView; + if (Build.VERSION.SDK_INT >= 16) { + mNotification.bigContentView = expandedView; + } + if (mNotification == null && prefPersistNotifications) { startForeground(NOTIFY_ID, mNotification); @@ -352,9 +397,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst stopTor(); - // Unregister all callbacks. - mCallbacks.kill(); - unregisterReceiver(mNetworkStateReceiver); } @@ -481,12 +523,12 @@ public class TorService extends Service implements TorServiceConstants, TorConst conn = null; } + else + killProcess(fileTor); - // killProcess(fileTor); killProcess(filePolipo); // killProcess(fileObfsclient); - } private void killProcess (File fileProcBin) throws IOException @@ -586,12 +628,35 @@ public class TorService extends Service implements TorServiceConstants, TorConst String TORRC_CONTROLPORT_FILE_KEY = "ControlPortWriteToFile"; fileControlPort = new File(appBinHome,"control.txt"); - extraLines.append(TORRC_CONTROLPORT_FILE_KEY).append(' ').append(fileControlPort.getCanonicalPath()); - extraLines.append('\n'); + extraLines.append(TORRC_CONTROLPORT_FILE_KEY).append(' ').append(fileControlPort.getCanonicalPath()).append('\n'); + + String socksPort = prefs.getString(TorConstants.PREF_SOCKS, TorServiceConstants.PORT_SOCKS_DEFAULT); + + String transPort = prefs.getString("pref_transport", TorServiceConstants.TOR_TRANSPROXY_PORT_DEFAULT+""); + String dnsPort = prefs.getString("pref_dnsport", TorServiceConstants.TOR_DNS_PORT_DEFAULT+""); + + extraLines.append("RunAsDaemon 1").append('\n'); + + extraLines.append("AvoidDiskWrites 1").append('\n'); + + + extraLines.append("SOCKSPort ").append(socksPort).append('\n'); + extraLines.append("SafeSocks 0").append('\n'); + extraLines.append("TestSocks 0").append('\n'); + extraLines.append("WarnUnsafeSocks 1").append('\n'); + + extraLines.append("TransPort ").append(transPort).append('\n'); + extraLines.append("DNSPort ").append(dnsPort).append('\n'); + extraLines.append("VirtualAddrNetwork 10.192.0.0/10").append('\n'); + extraLines.append("AutomapHostsOnResolve 1").append('\n'); + + extraLines.append(prefs.getString("pref_custom_torrc", "")); logNotice("updating torrc custom configuration..."); + debug("torrc.custom=" + extraLines.toString()); + File fileTorRcCustom = new File(fileTorRc.getAbsolutePath() + ".custom"); boolean success = installer.updateTorConfigCustom(fileTorRcCustom, extraLines.toString()); @@ -625,7 +690,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst } - private void updateSettings () + private void updateSettings () throws TimeoutException, IOException { SharedPreferences prefs = TorServiceUtils.getSharedPrefs(getApplicationContext()); @@ -638,6 +703,8 @@ public class TorService extends Service implements TorServiceConstants, TorConst Log.i(TAG,"debug logging:" + ENABLE_DEBUG_LOG); prefPersistNotifications = prefs.getBoolean(TorConstants.PREF_PERSIST_NOTIFICATIONS, true); + + updateTorConfigFile(); } public void initTor () throws Exception @@ -704,30 +771,29 @@ public class TorService extends Service implements TorServiceConstants, TorConst int code = 0; // Default state is "okay" - logMessage ("Transparent Proxying: clearing existing rules..."); + debug ("Transparent Proxying: clearing existing rules..."); //clear rules first mTransProxy.clearTransparentProxyingAll(this); if(proxyAll) { - showToolbarNotification(getString(R.string.setting_up_full_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); + // showToolbarNotification(getString(R.string.setting_up_full_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); code = mTransProxy.setTransparentProxyingAll(this); } else { - showToolbarNotification(getString(R.string.setting_up_app_based_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); + //showToolbarNotification(getString(R.string.setting_up_app_based_transparent_proxying_), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); code = mTransProxy.setTransparentProxyingByApp(this,AppManager.getApps(this, TorServiceUtils.getSharedPrefs(getApplicationContext()))); } - logMessage ("TorTransProxy resp code: " + code); + debug ("TorTransProxy resp code: " + code); if (code == 0) { - showToolbarNotification(getString(R.string.transparent_proxying_enabled), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); if (enableTether) { @@ -736,6 +802,11 @@ public class TorService extends Service implements TorServiceConstants, TorConst mTransProxy.enableTetheringRules(this); } + else + { + showToolbarNotification(getString(R.string.transparent_proxying_enabled), TRANSPROXY_NOTIFY_ID, R.drawable.ic_stat_tor); + + } } else { @@ -755,7 +826,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst private boolean disableTransparentProxy () throws Exception { - logMessage ("Transparent Proxying: disabling..."); + debug ("Transparent Proxying: disabling..."); if (mTransProxy == null) mTransProxy = new TorTransProxy(this, fileXtables); @@ -944,12 +1015,12 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (ENABLE_DEBUG_LOG) { - File fileLog = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS),"orbot-control-log.txt"); - PrintWriter pr = new PrintWriter(new FileWriter(fileLog,true)); - conn.setDebugging(pr); + //File fileLog = new File(getFilesDir(),"orbot-control-log.txt"); + //PrintWriter pr = new PrintWriter(new FileWriter(fileLog,true)); + //conn.setDebugging(pr); - File fileLog2 = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS),"orbot-tor-log.txt"); - conn.setConf("Log", "info file " + fileLog2.getCanonicalPath()); + File fileLog2 = new File(getFilesDir(),"orbot-tor-log.txt"); + conn.setConf("Log", "debug file " + fileLog2.getCanonicalPath()); } @@ -968,7 +1039,8 @@ public class TorService extends Service implements TorServiceConstants, TorConst catch (Exception ce) { conn = null; - logException( "Error connecting to Tor local control port",ce); + logException( "Error connecting to Tor local control port: " + ce.getMessage(),ce); + } try { @@ -1024,21 +1096,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst return result; } - private void checkAddressAndCountry () throws IOException - { - - if (TorService.ENABLE_DEBUG_LOG) - { - String torExternalAddress = conn.getInfo("address"); - String torCountry = conn.getInfo("ip-to-country/" + torExternalAddress); - - Log.d(TAG,"external address=" + torExternalAddress); - Log.d(TAG,"external country=" + torCountry); - - } - - } - /* private void getTorStatus () throws IOException { @@ -1104,36 +1161,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst return TorServiceConstants.PORT_HTTP; } - /** - * Returns the port number that the SOCKS proxy is running on - */ - public int getSOCKSPort() throws RemoteException { - - int socksPort = -1; - - SharedPreferences prefs = TorServiceUtils.getSharedPrefs(getApplicationContext()); - - String socksConfig = prefs.getString(TorConstants.PREF_SOCKS, TorServiceConstants.PORT_SOCKS_DEFAULT); - - try - { - - if (!socksConfig.equalsIgnoreCase("auto")) - if (socksConfig.contains(":")) - socksPort = Integer.parseInt(socksConfig.split(":")[1]); - else - socksPort = Integer.parseInt(socksConfig); - - } - catch (Exception e) - { - logException ("unable to parse socks config: " + socksConfig,e); - - } - - return socksPort; - } - @@ -1258,7 +1285,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst mTotalTrafficWritten += written; mTotalTrafficRead += read; - sendCallbackStatusMessage(written, read, mTotalTrafficWritten, mTotalTrafficRead); } @@ -1278,39 +1304,6 @@ public class TorService extends Service implements TorServiceConstants, TorConst //return count+" kB"; } - /* - class TotalUpdaterRunnable implements Runnable - { - - public void run () - { - - while (currentStatus != STATUS_OFF) - { - try - { - try - { - mTotalTrafficWritten = Long.parseLong(conn.getInfo("traffic/written")); - mTotalTrafficRead = Long.parseLong(conn.getInfo("traffic/read")); - - } - catch (Exception ioe) - { - Log.e(TAG,"error reading control port traffic",ioe); - } - - Thread.sleep(3000); //wait three seconds - } - catch (Exception e) - { - //nada - } - } - } - - }*/ - public void circuitStatus(String status, String circID, String path) { @@ -1322,14 +1315,34 @@ public class TorService extends Service implements TorServiceConstants, TorConst sb.append(": "); StringTokenizer st = new StringTokenizer(path,","); - String node = null; - String nodeName = null; + Node node = null; while (st.hasMoreTokens()) { - node = st.nextToken(); - nodeName = parseNodeName(node); - sb.append(nodeName); + String nodePath = st.nextToken(); + node = new Node(); + + String[] nodeParts; + + if (nodePath.contains("=")) + nodeParts = nodePath.split("="); + else + nodeParts = nodePath.split("~"); + + if (nodeParts.length == 1) + { + node.id = nodeParts[0].substring(1); + node.name = node.id; + } + else if (nodeParts.length == 2) + { + node.id = nodeParts[0].substring(1); + node.name = nodeParts[1]; + } + + node.status = status; + + sb.append(node.name); if (st.hasMoreTokens()) sb.append (" > "); @@ -1338,57 +1351,78 @@ public class TorService extends Service implements TorServiceConstants, TorConst logNotice(sb.toString()); //get IP from last nodename - if(status.equals("BUILT") && currentStatus==STATUS_ON){ - new getExternalIP().execute(nodeName); + if(status.equals("BUILT")){ + new getExternalIP().execute(node); + + hmBuiltNodes.put(node.id, node); + } + + if (status.equals("CLOSED")) + { + hmBuiltNodes.remove(node.id); + } } - private class getExternalIP extends AsyncTask<String, Void, String>{ + private HashMap<String,Node> hmBuiltNodes = new HashMap<String,Node>(); + + class Node + { + String status; + String id; + String name; + String ipAddress; + String country; + String organization; + } + + private class getExternalIP extends AsyncTask<Node, Void, Void>{ - private long time; @Override - protected String doInBackground(String... params) { - time = System.nanoTime(); - try { - String nodeDetails = conn.getInfo("ns/name/"+params[0]); - if (ENABLE_DEBUG_LOG) - { - Log.d(TAG,"Node Details: "+nodeDetails); - sendCallbackLogMessage("Node Details: "+nodeDetails); + protected Void doInBackground(Node... nodes) { + + if (conn != null) + { + try { + //String nodeDetails = conn.getInfo("ns/id/"+nodes[0].id); + Proxy proxy = new Proxy(Proxy.Type.HTTP, new InetSocketAddress("127.0.0.1", 8118)); - } - return nodeDetails; - } catch (TorControlError e) { - Log.d(TorService.TAG,"Error getting node details for: " + params[0]); - } catch (Exception e) { - Log.e(TorService.TAG,"Error getting node details",e); - } - return null; - } - - @Override - protected void onPostExecute(String result) { - // check if we need to update the exit IP - if(time > exitIPTime) { - exitIPTime = time; - - Pattern pattern = Pattern.compile(IPADDRESS_PATTERN); - Matcher matcher = null; - if(result!=null){ + URLConnection conn = new URL("https://onionoo.torproject.org/details?lookup=" + nodes[0].id).openConnection(proxy); + // getting JSON string from URL - sendCallbackLogMessage("Node Info: " + result); - - matcher = pattern.matcher(result); - if (matcher.find()) { - exitIP = matcher.group(); - sendCallbackLogMessage("Exit IP: "+exitIP); + StringBuffer json = new StringBuffer(); + String line = null; + BufferedReader reader = new BufferedReader(new InputStreamReader(conn.getInputStream())); + while ((line = reader.readLine())!=null) + json.append(line); + + JSONObject jsonNodeInfo = new org.json.JSONObject(json.toString()); + + JSONArray jsonRelays = jsonNodeInfo.getJSONArray("relays"); + if (jsonRelays.length() > 0) + { + nodes[0].ipAddress = jsonRelays.getJSONObject(0).getJSONArray("or_addresses").getString(0).split(":")[0]; + + nodes[0].country = jsonRelays.getJSONObject(0).getString("country_name"); + nodes[0].organization = jsonRelays.getJSONObject(0).getString("as_name"); + + + } + + return null; + + } catch (Exception e) { + + logException ("Error getting node details from onionoo",e); } } - } + return null; + } + } @@ -1432,28 +1466,12 @@ public class TorService extends Service implements TorServiceConstants, TorConst } - - /** - * This is a list of callbacks that have been registered with the - * service. Note that this is package scoped (instead of private) so - * that it can be accessed more efficiently from inner classes. - */ - final RemoteCallbackList<ITorServiceCallback> mCallbacks - = new RemoteCallbackList<ITorServiceCallback>(); - /** * The IRemoteInterface is defined through IDL */ private final ITorService.Stub mBinder = new ITorService.Stub() { - - public void registerCallback(ITorServiceCallback cb) { - if (cb != null) mCallbacks.register(cb); - } - public void unregisterCallback(ITorServiceCallback cb) { - if (cb != null) mCallbacks.unregister(cb); - } public int getStatus () { return getTorStatus(); } @@ -1607,13 +1625,15 @@ public class TorService extends Service implements TorServiceConstants, TorConst { public void run () { - try { conn.signal("NEWNYM"); + try { + + conn.signal("NEWNYM"); //checkAddressAndCountry(); } catch (IOException ioe){ - logMessage("error requesting newnym: " + ioe.getLocalizedMessage()); + debug("error requesting newnym: " + ioe.getLocalizedMessage()); } } }.start(); @@ -1632,7 +1652,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst for (String value : configBuffer) { - logMessage("removing torrc conf: " + value); + debug("removing torrc conf: " + value); } @@ -1647,7 +1667,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst for (String value : configBuffer) { - logMessage("Setting torrc conf: " + value); + debug("Setting torrc conf: " + value); } @@ -1674,111 +1694,59 @@ public class TorService extends Service implements TorServiceConstants, TorConst return false; } + + @Override + public String[] getStatusMessage() throws RemoteException { + + synchronized (mStatusBuffer) + { + String[] status = mStatusBuffer.toArray(new String[mStatusBuffer.size()]); + mStatusBuffer.clear(); + return status; + } + + } + + @Override + public String[] getLog() throws RemoteException { + + synchronized (mLogBuffer) + { + String[] status = mLogBuffer.toArray(new String[mLogBuffer.size()]); + mLogBuffer.clear(); + return status; + } + } + + public long[] getBandwidth() throws RemoteException { + + long[] bw = {lastRead,lastWritten,mTotalTrafficRead,mTotalTrafficWritten}; + return bw; + } }; - - private synchronized void sendCallbackStatusMessage (String newStatus) + private ArrayList<String> mStatusBuffer = new ArrayList<String>(); + + private void sendCallbackStatusMessage (String newStatus) { - - if (mCallbacks == null) - return; - - // Broadcast to all clients the new value. - final int N = mCallbacks.beginBroadcast(); - - inCallback = true; - - if (N > 0) - { - for (int i=0; i<N; i++) { - try { - mCallbacks.getBroadcastItem(i).statusChanged(newStatus); - - - } catch (RemoteException e) { - // The RemoteCallbackList will take care of removing - // the dead object for us. - } - } - } - - mCallbacks.finishBroadcast(); - inCallback = false; + mStatusBuffer.add(newStatus); } - private synchronized void sendCallbackStatusMessage (long upload, long download, long written, long read) + private void sendCallbackStatusMessage (long upload, long download, long written, long read) { - if (mCallbacks == null) - return; - // Broadcast to all clients the new value. - final int N = mCallbacks.beginBroadcast(); - - inCallback = true; - - if (N > 0) - { - for (int i=0; i<N; i++) { - try { - mCallbacks.getBroadcastItem(i).updateBandwidth(upload, download, written, read); - - } catch (RemoteException e) { - // The RemoteCallbackList will take care of removing - // the dead object for us. - } - } - } - - mCallbacks.finishBroadcast(); - inCallback = false; } + private ArrayList<String> mLogBuffer = new ArrayList<String>(); - private synchronized void sendCallbackLogMessage (String logMessage) + + private void sendCallbackLogMessage (String logMessage) { - if (mCallbacks == null) - return; - callbackBuffer.add(logMessage); - - if (!inCallback) - { + mLogBuffer.add(logMessage); - inCallback = true; - // Broadcast to all clients the new value. - final int N = mCallbacks.beginBroadcast(); - - - if (N > 0) - { - - Iterator<String> it = callbackBuffer.iterator(); - String status = null; - - while (it.hasNext()) - { - status = it.next(); - - for (int i=0; i<N; i++) { - try { - mCallbacks.getBroadcastItem(i).logMessage(status); - - } catch (RemoteException e) { - // The RemoteCallbackList will take care of removing - // the dead object for us. - } - } - } - - callbackBuffer.clear(); - } - - mCallbacks.finishBroadcast(); - inCallback = false; - } - } /* @@ -1844,6 +1812,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst SharedPreferences prefs = TorServiceUtils.getSharedPrefs(getApplicationContext()); + /* String socksConfig = prefs.getString(TorConstants.PREF_SOCKS, TorServiceConstants.PORT_SOCKS_DEFAULT); enableSocks (socksConfig,false); @@ -1852,7 +1821,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst String dnsPort = prefs.getString("pref_dnsport", TorServiceConstants.TOR_DNS_PORT_DEFAULT+""); enableTransProxyAndDNSPorts(transPort, dnsPort); - + */ boolean useBridges = prefs.getBoolean(TorConstants.PREF_BRIDGES_ENABLED, false); @@ -1933,7 +1902,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst if (useBridges) { - logMessage ("Using bridges"); + debug ("Using bridges"); String bridgeCfgKey = "Bridge"; String bridgeList = prefs.getString(TorConstants.PREF_BRIDGES_LIST,null); @@ -1943,7 +1912,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst String msgBridge = getString(R.string.bridge_requires_ip) + getString(R.string.send_email_for_bridges); showToolbarNotification(msgBridge, ERROR_NOTIFY_ID, R.drawable.ic_stat_tor); - logMessage(msgBridge); + debug(msgBridge); return false; } @@ -1962,7 +1931,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst while (st.hasMoreTokens()) { String bridgeConfigLine = st.nextToken().trim(); - logMessage("Adding bridge: " + bridgeConfigLine); + debug("Adding bridge: " + bridgeConfigLine); mBinder.updateConfiguration(bridgeCfgKey, bridgeConfigLine, false); } @@ -1974,13 +1943,13 @@ public class TorService extends Service implements TorServiceConstants, TorConst { String bridgeConfig = "obfs2,obfs3,scramblesuit exec " + fileObfsclient.getCanonicalPath(); - logMessage ("Using OBFUSCATED bridges: " + bridgeConfig); + debug ("Using OBFUSCATED bridges: " + bridgeConfig); mBinder.updateConfiguration("ClientTransportPlugin",bridgeConfig, false); } else { - logMessage ("Using standard bridges"); + debug ("Using standard bridges"); } @@ -2077,7 +2046,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst String hsDirPath = new File(appCacheHome,"hs" + hsPort).getCanonicalPath(); - logMessage("Adding hidden service on port: " + hsPortConfig); + debug("Adding hidden service on port: " + hsPortConfig); mBinder.updateConfiguration("HiddenServiceDir",hsDirPath, false); @@ -2104,6 +2073,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst return true; } + /* private void enableSocks (String socks, boolean safeSocks) throws RemoteException { mBinder.updateConfiguration("SOCKSPort", socks, false); @@ -2123,9 +2093,7 @@ public class TorService extends Service implements TorServiceConstants, TorConst mBinder.updateConfiguration("VirtualAddrNetwork","10.192.0.0/10",false); mBinder.updateConfiguration("AutomapHostsOnResolve","1",false); mBinder.saveConfiguration(); - - - } + }*/ private void blockPlaintextPorts (String portList) throws RemoteException { diff --git a/src/org/torproject/android/service/TorTransProxy.java b/src/org/torproject/android/service/TorTransProxy.java index bf3efbd..db97777 100644 --- a/src/org/torproject/android/service/TorTransProxy.java +++ b/src/org/torproject/android/service/TorTransProxy.java @@ -497,7 +497,7 @@ public class TorTransProxy implements TorServiceConstants { private void logMessage (String msg) { if (mTorService != null) - mTorService.logMessage(msg); + mTorService.debug(msg); } public int clearTransparentProxyingAll(Context context) throws Exception

1 0

[orbot/master] remove external storage perm (debug log written internally)
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit 498f6472bfa4726fad5896d703c7c7f87f8a7fec Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 09:20:09 2014 -0400 remove external storage perm (debug log written internally) need to add ability to share internal file after marking it global read --- AndroidManifest.xml | 1 - 1 file changed, 1 deletion(-) diff --git a/AndroidManifest.xml b/AndroidManifest.xml index 4f1898b..4efb11c 100644 --- a/AndroidManifest.xml +++ b/AndroidManifest.xml @@ -13,7 +13,6 @@ <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED"/> <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" /> <uses-permission android:name="android.permission.ACCESS_SUPERUSER"/> - <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/> <uses-permission android:name="org.torproject.android.MANAGE_TOR"/> <application android:name=".OrbotApp" android:icon="@drawable/ic_launcher"

1 0

[orbot/master] update to 14.0.5-beta
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit d439176feac4eec5a3f72bf3254d3ca369001319 Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 13:39:16 2014 -0400 update to 14.0.5-beta --- AndroidManifest.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/AndroidManifest.xml b/AndroidManifest.xml index 4efb11c..f2256fc 100644 --- a/AndroidManifest.xml +++ b/AndroidManifest.xml @@ -1,8 +1,8 @@ <?xml version="1.0" encoding="utf-8"?> <manifest xmlns:android="http://schemas.android.com/apk/res/android" package="org.torproject.android" - android:versionName="14.0.4.3" - android:versionCode="109" + android:versionName="14.0.5-beta" + android:versionCode="110" android:installLocation="auto" >

1 0

[orbot/master] updating ant build shell permissions
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit 1e6ff3579b1b998cd15f37fe8a775b7f9c42c058 Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 10:48:37 2014 -0400 updating ant build shell permissions --- 0 files changed diff --git a/update-ant-build.sh b/update-ant-build.sh old mode 100644 new mode 100755

1 0

[orbot/master] update CHANGELOG
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit 8cc8886b23eb9cf146a49857cc1f3f8af219c206 Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 13:39:38 2014 -0400 update CHANGELOG --- CHANGELOG | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/CHANGELOG b/CHANGELOG index 5c03e2b..43f8ab4 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,5 +1,22 @@ NOTE: Specific #s below correspond to Trac tickets logged and maintained at https://trac.torproject.org/projects/tor/ + +d439176 update to 14.0.5-beta +c4cf845 handle NPE in case service disconnects +1e6ff35 updating ant build shell permissions +498f647 remove external storage perm (debug log written internally) need to add ability to share internal file after marking it global +e1ec776 re-arch status callbacks and improve notification +c7409b7 update commit pegs for external +f2f076c fixes for layout and notification data be sure to only get the IP of the last hop (that is the exit) +a6da4bf small cleanup of asynctask and error handling +181b922 make the notification larger for more display room! +a82d2e4 remove unnecessary jar file +3bc8760 Merge branch 'new_notification_expanded' of github.com:amoghbl1/orbot-1 into amoghbl1-new_notification_expanded +41064d8 add persistent flag +b4079f4 update transifex config +8a934c9 don't close socket, and use HALT for shutdown +f9b38e6 if service is null, please start it +a447bbf update strings 522f293 update to 14.0.4.3 70615d6 fixes for status, service, binding and start/stop fbc1a87 update to 14.0.5-alpha

1 0

[orbot/master] handle NPE in case service disconnects
by n8fr8＠torproject.org 14 Jul '14

14 Jul '14

commit c4cf845650aee14ef2bca637937f253fc13701b5 Author: Nathan Freitas <nathan(a)freitas.net> Date: Mon Jul 14 13:16:19 2014 -0400 handle NPE in case service disconnects --- src/org/torproject/android/Orbot.java | 60 ++++++++++----------------------- 1 file changed, 18 insertions(+), 42 deletions(-) diff --git a/src/org/torproject/android/Orbot.java b/src/org/torproject/android/Orbot.java index 06ffe57..2c63711 100644 --- a/src/org/torproject/android/Orbot.java +++ b/src/org/torproject/android/Orbot.java @@ -1082,22 +1082,25 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic mHandler.sendMessage(msg); } - long[] bws = mService.getBandwidth(); - Message msg = mHandler.obtainMessage(TorServiceConstants.MESSAGE_TRAFFIC_COUNT); - msg.getData().putLong("download", bws[0]); - msg.getData().putLong("upload", bws[1]); - msg.getData().putLong("readTotal", bws[2]); - msg.getData().putLong("writeTotal", bws[3]); - mHandler.sendMessage(msg); - - try { Thread.sleep(1000); } - catch (Exception e){} - - - torStatus = mService.getStatus(); + if (mService != null) + { + long[] bws = mService.getBandwidth(); + Message msg = mHandler.obtainMessage(TorServiceConstants.MESSAGE_TRAFFIC_COUNT); + msg.getData().putLong("download", bws[0]); + msg.getData().putLong("upload", bws[1]); + msg.getData().putLong("readTotal", bws[2]); + msg.getData().putLong("writeTotal", bws[3]); + mHandler.sendMessage(msg); + + try { Thread.sleep(1000); } + catch (Exception e){} + } + + if (mService != null) + torStatus = mService.getStatus(); } } - catch (RemoteException re) + catch (Exception re) { Log.e(TAG, "error getting service updates",re); } @@ -1230,37 +1233,10 @@ public class Orbot extends ActionBarActivity implements TorConstants, OnLongClic mKeepUpdating = false; mService = null; Log.d(TAG,"service was disconnected"); + } }; - - /* - private void createProgressDialog (String msg) - { - if (progressDialog != null && progressDialog.isShowing()) - { - progressDialog.setMessage(msg); - } - else - { - progressDialog = ProgressDialog.show(Orbot.this, "", msg); - progressDialog.setCancelable(true); - } - } - - private void hideProgressDialog () - { - - if (progressDialog != null && progressDialog.isShowing()) - { - progressDialog.dismiss(); - progressDialog = null; - } - - } - */ - - private void setLocale () {

1 0