From 11528c3f7124090ef44675cf613a949ed07ad9bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Parm=C3=A9nides=20GV?= Date: Tue, 14 Oct 2014 18:39:03 +0200 Subject: Start EIP after downloading an authed certificate. Always. Previously we didn't do so if EIP hadn't been started before. Fix a small bug introduced during latest refactoring, authentication did not work because the final result variable wasn't assigned. --- app/src/debug/java/se/leap/bitmaskclient/ProviderAPI.java | 2 +- app/src/main/java/se/leap/bitmaskclient/Dashboard.java | 6 ++---- app/src/release/java/se/leap/bitmaskclient/ProviderAPI.java | 2 +- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/app/src/debug/java/se/leap/bitmaskclient/ProviderAPI.java b/app/src/debug/java/se/leap/bitmaskclient/ProviderAPI.java index b83f33a1..f51099a3 100644 --- a/app/src/debug/java/se/leap/bitmaskclient/ProviderAPI.java +++ b/app/src/debug/java/se/leap/bitmaskclient/ProviderAPI.java @@ -260,7 +260,7 @@ public class ProviderAPI extends IntentService { if(validUserLoginData(username, password)) { String server = (String) task.get(Provider.API_URL); - authenticate(username, password, server); + result = authenticate(username, password, server); broadcast_progress(progress++); } else { if(!wellFormedPassword(password)) { diff --git a/app/src/main/java/se/leap/bitmaskclient/Dashboard.java b/app/src/main/java/se/leap/bitmaskclient/Dashboard.java index 761afc0a..16e4cf03 100644 --- a/app/src/main/java/se/leap/bitmaskclient/Dashboard.java +++ b/app/src/main/java/se/leap/bitmaskclient/Dashboard.java @@ -65,6 +65,7 @@ public class Dashboard extends Activity implements LogInDialog.LogInDialogInterf public static final String START_ON_BOOT = "dashboard start on boot"; final public static String ON_BOOT = "dashboard on boot"; public static final String APP_VERSION = "bitmask version"; + final public static String TAG = Dashboard.class.getSimpleName(); private EipServiceFragment eipFragment; @@ -498,10 +499,7 @@ public class Dashboard extends Activity implements LogInDialog.LogInDialogInterf changeStatusMessage(resultCode); if(mProgressBar != null) mProgressBar.setVisibility(ProgressBar.GONE); - if(EipServiceFragment.isEipSwitchChecked()) - eipStart(); - else - eipStatus.setText(R.string.eip_state_not_connected); + eipStart(); } else if(resultCode == ProviderAPI.INCORRECTLY_DOWNLOADED_CERTIFICATE) { setResult(RESULT_CANCELED); changeStatusMessage(resultCode); diff --git a/app/src/release/java/se/leap/bitmaskclient/ProviderAPI.java b/app/src/release/java/se/leap/bitmaskclient/ProviderAPI.java index 3c8ec607..549eee0d 100644 --- a/app/src/release/java/se/leap/bitmaskclient/ProviderAPI.java +++ b/app/src/release/java/se/leap/bitmaskclient/ProviderAPI.java @@ -255,7 +255,7 @@ public class ProviderAPI extends IntentService { String server = (String) task.get(Provider.API_URL); - authenticate(username, password, server); + result = authenticate(username, password, server); broadcast_progress(progress++); } else { if(!wellFormedPassword(password)) { -- cgit v1.2.3