From 66db5aa79d0f44e0196a459b873379c666b78c48 Mon Sep 17 00:00:00 2001 From: Billy Brawner Date: Tue, 16 Oct 2018 07:25:28 -0500 Subject: [PATCH] Fix error reports and allow disabling them --- app/acra.properties.sample | 6 +++--- app/src/main/assets/Privacy Policy.md | 18 +++++------------- .../simplemarkdown/MarkdownApplication.java | 2 -- .../view/activity/ExplorerActivity.java | 6 ------ app/src/main/res/values/strings.xml | 6 ++++++ app/src/main/res/xml/pref_general.xml | 9 ++++++++- 6 files changed, 22 insertions(+), 25 deletions(-) diff --git a/app/acra.properties.sample b/app/acra.properties.sample index bc9f4f5..c0c912f 100644 --- a/app/acra.properties.sample +++ b/app/acra.properties.sample @@ -1,3 +1,3 @@ -acraUrl=ACRA_URL -acraUser=ACRA_USER -acraPass=ACRA_PASS +url=ACRA_URL +user=ACRA_USER +pass=ACRA_PASS diff --git a/app/src/main/assets/Privacy Policy.md b/app/src/main/assets/Privacy Policy.md index c05498b..1d0806e 100644 --- a/app/src/main/assets/Privacy Policy.md +++ b/app/src/main/assets/Privacy Policy.md @@ -1,30 +1,22 @@ ## Privacy Policy -First and foremost, Simple Markdown DOES NOT collect any personally identifiable information. The -internet access permission is requested primarily for retrieving images from the internet in case -you embed them in your markdown, but it also allows me to send automated error and crash reports -to myself whenever the app runs into an issue. These error reports are powered by [ACRA](https://github.com/ACRA/acra), which is -an open source error reporting solution, and are sent to a private server that only I have access -to. These error reports are used exclusively for fixing problems that occur while you're using the -app, and contain only the bare minimum information that I need to be able to resolve the issue. The -information sent may include: +First and foremost, Simple Markdown DOES NOT collect any personally identifiable information. The internet access permission is requested primarily for retrieving images from the internet in case you embed them in your markdown, but it also allows me to send automated error and crash reports to myself whenever the app runs into an issue. These error reports are powered by [ACRA](https://github.com/ACRA/acra), which is an open source error reporting solution, and are sent to a private server that only I have access to. These error reports are used exclusively for fixing problems that occur while you're using the app, and contain only the bare minimum information that I need to be able to resolve the issue. The information sent may include the following: - the version of Android your device is running (e.g. 7.1) - details about the version of SimpleMarkdown that you are using (Google Play version, Samsung Galaxy Apps version, FDroid version, version number, etc) -- any logs that SimpleMarkdown (but not other apps) might have created during execution - the app identifier (com.wbrawner.simplemarkdown) - your device's manufacturer (Samsung, Huawei, LG, etc) - your device's model (Galaxy S8, Mate 10 pro, G7, etc) -- your device's configuration at app start and at the moment of the crash (the configuration tells me things like if you had the keyboard open, and if you were using the app in landscape or portrait mode. See the end of this privacy policy for a sample of that this might look like.) +- your device's configuration at app start and at the moment of the error (the configuration tells me things like if you had the keyboard open, and if you were using the app in landscape or portrait mode. See the end of this privacy policy for a sample of that this might look like.) - an estimation of the amount of available memory your device has (7353192448) - an estimation of the amount of total memory your device has (55540875264) - a stacktrace of the error (a stacktrace tells me which part of my code caused the error, what kind of error it was, and which parts of the code ran up to that error. See the end of this privacy policy for a sample of what this might look like.) - any custom settings you have set within SimpleMarkdown (your default launch screen, default file directory, etc. Note that this does NOT include settings specific to your device, like whether or not you have WiFi enabled for example) -- details about which thread the crash occurred on (e.g. main, file handling, network) +- details about which thread the error occurred on (e.g. main, file handling, network) - the time and date you started the app -- the time and date the crash occurred +- the time and date the error occurred -This information is the only information that I collect, and it's strictly used for finding and fixing issues in the code as quickly as possible. If you don't feel comfortable with this however, you are able to opt-out from the settings menu. Should you choose to opt-out of automated crash reports, I would very much appreciate it if you could contact me when you run into an issue either by email: [support@wbrawner.com](mailto:support@wbrawner.com) or by submitting an issue via the [GitHub page](https://github.com/wbrawner/SimpleMarkdown). +This information is the only information that I collect, and it's strictly used for finding and fixing issues in the code as quickly as possible. If you don't feel comfortable with this however, you are able to opt-out from the settings menu. Should you choose to opt-out of automated error reports, I would very much appreciate it if you could contact me when you run into an issue either by email: [support@wbrawner.com](mailto:support@wbrawner.com) or by submitting an issue via the [GitHub page](https://github.com/wbrawner/SimpleMarkdown). ### Sample data diff --git a/app/src/main/java/com/wbrawner/simplemarkdown/MarkdownApplication.java b/app/src/main/java/com/wbrawner/simplemarkdown/MarkdownApplication.java index f20e7c7..4152d1e 100644 --- a/app/src/main/java/com/wbrawner/simplemarkdown/MarkdownApplication.java +++ b/app/src/main/java/com/wbrawner/simplemarkdown/MarkdownApplication.java @@ -13,7 +13,6 @@ import static com.wbrawner.simplemarkdown.BuildConfig.ACRA_PASS; import static com.wbrawner.simplemarkdown.BuildConfig.ACRA_URL; import static com.wbrawner.simplemarkdown.BuildConfig.ACRA_USER; import static org.acra.ReportField.ANDROID_VERSION; -import static org.acra.ReportField.APPLICATION_LOG; import static org.acra.ReportField.APP_VERSION_CODE; import static org.acra.ReportField.APP_VERSION_NAME; import static org.acra.ReportField.AVAILABLE_MEM_SIZE; @@ -38,7 +37,6 @@ import static org.acra.ReportField.USER_CRASH_DATE; ANDROID_VERSION, APP_VERSION_CODE, APP_VERSION_NAME, - APPLICATION_LOG, AVAILABLE_MEM_SIZE, BRAND, BUILD_CONFIG, diff --git a/app/src/main/java/com/wbrawner/simplemarkdown/view/activity/ExplorerActivity.java b/app/src/main/java/com/wbrawner/simplemarkdown/view/activity/ExplorerActivity.java index 59b2906..1cafa80 100644 --- a/app/src/main/java/com/wbrawner/simplemarkdown/view/activity/ExplorerActivity.java +++ b/app/src/main/java/com/wbrawner/simplemarkdown/view/activity/ExplorerActivity.java @@ -122,12 +122,6 @@ public class ExplorerActivity extends AppCompatActivity { ACRA.getErrorReporter().handleException(e, false); updateListView(); menu.findItem(R.id.action_use_sdcard).setVisible(false); - // TODO: Report this? -// Crashlytics.logException(e); -// Crashlytics.log(mounts.length + " mounts found"); -// for (File mount : mounts) { -// Crashlytics.log(mount.getAbsolutePath()); -// } } if (sdcardSelected) { menu.findItem(R.id.action_use_sdcard).setChecked(true); diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 016c2e0..fba16cc 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -42,6 +42,12 @@ Select \u2191 Go up Privacy + acra.enable + Enable automated error reports + Error reports will not be sent + Error reports will be sent + Files will automatically save + Files will not be automatically saved @string/value_edit_view @string/value_explorer_view diff --git a/app/src/main/res/xml/pref_general.xml b/app/src/main/res/xml/pref_general.xml index 0a3ff1c..a51d022 100644 --- a/app/src/main/res/xml/pref_general.xml +++ b/app/src/main/res/xml/pref_general.xml @@ -3,7 +3,8 @@ +