Merge pull request #2798 from element-hq/feature/bma/externalPermalinkFix

Feature/bma/external permalink fix

app/src/main/AndroidManifest.xml

@@ -84,7 +84,13 @@
                 <category android:name="android.intent.category.BROWSABLE" />
 
                 <data android:scheme="https" />
-                <data android:host="*.element.io" />
+                <!-- Note: we can't use "*.element.io" here because it'll intercept the "mas.element.io" domain too. -->
+                <!-- Matching asset file: https://app.element.io/.well-known/assetlinks.json -->
+                <data android:host="app.element.io" />
+                <!-- Matching asset file: https://develop.element.io/.well-known/assetlinks.json -->
+                <data android:host="develop.element.io" />
+                <!-- Matching asset file: https://staging.element.io/.well-known/assetlinks.json -->
+                <data android:host="staging.element.io" />
             </intent-filter>
             <!--
               matrix.to links

app/src/main/kotlin/io/element/android/x/MainActivity.kt

@@ -39,7 +39,6 @@ import io.element.android.compound.theme.ElementTheme
 import io.element.android.compound.theme.Theme
 import io.element.android.compound.theme.isDark
 import io.element.android.compound.theme.mapToTheme
-import io.element.android.features.call.ui.ElementCallActivity
 import io.element.android.features.lockscreen.api.handleSecureFlag
 import io.element.android.features.lockscreen.api.isLocked
 import io.element.android.libraries.architecture.bindings
@@ -59,13 +58,6 @@ class MainActivity : NodeActivity() {
         Timber.tag(loggerTag.value).w("onCreate, with savedInstanceState: ${savedInstanceState != null}")
         installSplashScreen()
         super.onCreate(savedInstanceState)
-        if (ElementCallActivity.maybeStart(this, intent)) {
-            Timber.tag(loggerTag.value).w("Starting Element Call Activity")
-            if (savedInstanceState == null) {
-                finish()
-                return
-            }
-        }
         appBindings = bindings()
         appBindings.lockScreenService().handleSecureFlag(this)
         enableEdgeToEdge()
@@ -149,12 +141,6 @@ class MainActivity : NodeActivity() {
     override fun onNewIntent(intent: Intent) {
         super.onNewIntent(intent)
         Timber.tag(loggerTag.value).w("onNewIntent")
-
-        if (ElementCallActivity.maybeStart(this, intent)) {
-            Timber.tag(loggerTag.value).w("Starting Element Call Activity")
-            return
-        }
-
         // If the mainNode is not init yet, keep the intent for later.
         // It can happen when the activity is killed by the system. The methods are called in this order :
         // onCreate(savedInstanceState=true) -> onNewIntent -> onResume -> onMainNodeInit

features/call/src/main/AndroidManifest.xml

@@ -16,8 +16,12 @@
 
 <manifest xmlns:android="http://schemas.android.com/apk/res/android">
 
-    <uses-feature android:name="android.hardware.camera" android:required="false" />
+    <uses-feature
-    <uses-feature android:name="android.hardware.microphone" android:required="false" />
+        android:name="android.hardware.camera"
+        android:required="false" />
+    <uses-feature
+        android:name="android.hardware.microphone"
+        android:required="false" />
 
     <uses-permission android:name="android.permission.RECORD_AUDIO" />
     <uses-permission android:name="android.permission.CAMERA" />
@@ -28,19 +32,27 @@
     <application>
         <activity
             android:name=".ui.ElementCallActivity"
-            android:label="@string/element_call"
-            android:exported="true"
-            android:taskAffinity="io.element.android.features.call"
             android:configChanges="screenSize|screenLayout|orientation|keyboardHidden|keyboard|navigation|uiMode"
-            android:launchMode="singleTask">
+            android:exported="true"
+            android:label="@string/element_call"
+            android:launchMode="singleTask"
+            android:taskAffinity="io.element.android.features.call">
 
-            <!--
+            <intent-filter android:autoVerify="true">
-             Note: intent-filter for https://call.element.io link is now managed by the MainActivity.
+                <action android:name="android.intent.action.VIEW" />
-            -->
 
+                <category android:name="android.intent.category.DEFAULT" />
+                <category android:name="android.intent.category.BROWSABLE" />
+
+                <data android:scheme="https" />
+
+                <!-- Matching asset file: https://call.element.io/.well-known/assetlinks.json -->
+                <data android:host="call.element.io" />
+            </intent-filter>
             <!-- Custom scheme to handle urls from other domains in the format: element://call?url=https%3A%2F%2Felement.io -->
             <intent-filter>
                 <action android:name="android.intent.action.VIEW" />
+
                 <category android:name="android.intent.category.DEFAULT" />
                 <category android:name="android.intent.category.BROWSABLE" />
 
@@ -50,6 +62,7 @@
             <!-- Custom scheme to handle urls from other domains in the format: io.element.call:/?url=https%3A%2F%2Felement.io -->
             <intent-filter>
                 <action android:name="android.intent.action.VIEW" />
+
                 <category android:name="android.intent.category.DEFAULT" />
                 <category android:name="android.intent.category.BROWSABLE" />
 
@@ -57,7 +70,10 @@
             </intent-filter>
 
         </activity>
-        <service android:name=".CallForegroundService" android:enabled="true" android:foregroundServiceType="mediaPlayback" />
+        <service
+            android:name=".CallForegroundService"
+            android:enabled="true"
+            android:foregroundServiceType="mediaPlayback" />
     </application>
 
 </manifest>

features/call/src/main/kotlin/io/element/android/features/call/ui/ElementCallActivity.kt

@@ -17,7 +17,6 @@
 package io.element.android.features.call.ui
 
 import android.Manifest
-import android.app.Activity
 import android.content.Context
 import android.content.Intent
 import android.content.Intent.FLAG_ACTIVITY_NEW_TASK
@@ -36,7 +35,6 @@ import androidx.compose.runtime.collectAsState
 import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
-import androidx.core.app.ActivityOptionsCompat
 import androidx.core.content.IntentCompat
 import com.bumble.appyx.core.integrationpoint.NodeComponentActivity
 import io.element.android.compound.theme.ElementTheme
@@ -49,7 +47,6 @@ import io.element.android.features.call.di.CallBindings
 import io.element.android.features.call.utils.CallIntentDataParser
 import io.element.android.features.preferences.api.store.AppPreferencesStore
 import io.element.android.libraries.architecture.bindings
-import io.element.android.libraries.core.bool.orFalse
 import javax.inject.Inject
 
 class ElementCallActivity : NodeComponentActivity(), CallScreenNavigator {
@@ -66,28 +63,6 @@ class ElementCallActivity : NodeComponentActivity(), CallScreenNavigator {
             }
             context.startActivity(intent)
         }
-
-        /**
-         * Starts the [ElementCallActivity] if the intent contains a valid URL,
-         * and returns true if it's the case.
-         */
-        fun maybeStart(
-            activity: Activity,
-            intent: Intent?,
-        ): Boolean {
-            return intent?.data
-                ?.takeIf { uri -> uri.scheme == "https" && uri.host == "call.element.io" }
-                ?.let { uri ->
-                    val callIntent = Intent(activity, ElementCallActivity::class.java).apply {
-                        data = uri
-                    }
-                    // Disable animation since MainActivity has already been animated.
-                    val options = ActivityOptionsCompat.makeCustomAnimation(activity, 0, 0)
-                    activity.startActivity(callIntent, options.toBundle())
-                    true
-                }
-                .orFalse()
-        }
     }
 
     @Inject lateinit var callIntentDataParser: CallIntentDataParser