Merge pull request #1535 from AllanWang/fix-biometrics

Move biometric init out of coroutine launch

app/src/main/kotlin/com/pitchedapps/frost/StartActivity.kt

@@ -70,8 +70,8 @@ class StartActivity : KauBaseActivity() {
             showInvalidWebView()
         }
 
-        launch {
         val authDefer = BiometricUtils.authenticate(this@StartActivity)
+        launch {
             try {
                 FbCookie.switchBackUser()
                 val cookies = ArrayList(cookieDao.selectAll())

app/src/main/kotlin/com/pitchedapps/frost/activities/WebOverlayActivity.kt

@@ -209,8 +209,8 @@ abstract class WebOverlayActivityBase : BaseActivity(),
 
         with(web) {
             Prefs.prevId = Prefs.userId
-            launch {
             val authDefer = BiometricUtils.authenticate(this@WebOverlayActivityBase)
+            launch {
                 if (userId != Prefs.userId)
                     FbCookie.switchUser(userId)
                 authDefer.await()

app/src/main/kotlin/com/pitchedapps/frost/utils/BiometricUtils.kt

@@ -65,6 +65,11 @@ object BiometricUtils {
         return Prefs.biometricsEnabled && System.currentTimeMillis() - lastUnlockTime > UNLOCK_TIME_INTERVAL
     }
 
+    /**
+     * Generates a prompt dialog and attempt to return an auth object.
+     * Note that the underlying request will call [androidx.fragment.app.FragmentTransaction.commit],
+     * so this cannot happen after onSaveInstanceState.
+     */
     fun authenticate(activity: FragmentActivity, force: Boolean = false): BiometricDeferred {
         val deferred: BiometricDeferred = CompletableDeferred()
         if (!force && !shouldPrompt(activity)) {

app/src/main/play/en-US/whatsnew

@@ -1,3 +1,4 @@
 v2.4.0
 
 * Removed web only mode for auth requests. Marking notifications as read is now disabled by default to deal with phishing accusations.
+* Save images with the correct extensions.

app/src/main/res/xml/frost_changelog.xml

@@ -8,7 +8,7 @@
 
     <version title="v2.4.0" />
     <item text="Removed web only mode for auth requests. Marking notifications as read is now disabled by default to deal with phishing accusations." />
-    <item text="" />
+    <item text="Save images with the correct extensions." />
     <item text="" />
 
     <version title="v2.3.2" />

docs/Changelog.md

@@ -2,6 +2,7 @@
 
 ## v2.4.0
 * Removed web only mode for auth requests. Marking notifications as read is now disabled by default to deal with phishing accusations.
+* Save images with the correct extensions.
 
 ## v2.3.2
 * Disable auto feed refresh by default and add setting to re-enable it