From 0708862abe8463a053282fa898a6249fd394544b Mon Sep 17 00:00:00 2001
From: Sylvain Berfini <sylvain.berfini@belledonne-communications.com>
Date: Thu, 30 Mar 2023 15:07:31 +0200
Subject: [PATCH] Prevent crash when clicking on URI in chat if not matching
 app is found

---
 CHANGELOG.md                                               | 1 +
 .../main/chat/fragments/DetailChatRoomFragment.kt          | 7 +++++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 500669cb6..f43443029 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -14,6 +14,7 @@ Group changes to describe their impact on the project, as follows:
 
 ### Fixed
 - Admin weren't visible for non admin users in group chat rooms
+- Crash when clicking on URI in chat if not matching app is found on Android to handle it
 - LIME update threshold wasn't set, causing a request to be made after each REGISTER
 
 ### Changed
diff --git a/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt b/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
index 00aa29fc6..40551ddd8 100644
--- a/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
+++ b/app/src/main/java/org/linphone/activities/main/chat/fragments/DetailChatRoomFragment.kt
@@ -505,14 +505,17 @@ class DetailChatRoomFragment : MasterFragment<ChatRoomDetailFragmentBinding, Cha
             viewLifecycleOwner
         ) {
             it.consume { url ->
+                val uri = Uri.parse(url)
                 val browserIntent = Intent(
                     Intent.ACTION_VIEW,
-                    Uri.parse(url)
+                    uri
                 )
                 try {
                     startActivity(browserIntent)
                 } catch (se: SecurityException) {
-                    Log.e("[Chat Room] Failed to start browser intent, $se")
+                    Log.e("[Chat Room] Failed to start browser intent from uri [$uri]: $se")
+                } catch (anfe: ActivityNotFoundException) {
+                    Log.e("[Chat Room] Failed to find app matching intent from uri [$uri]: $anfe")
                 }
             }
         }