fix(*): remove group from parsing as it made Android API 21 crashes

apivideo · Sep 27, 2023 · 5b4842c · 5b4842c
1 parent 169c1a2
commit 5b4842c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/build.gradle b/build.gradle
@@ -4,7 +4,7 @@ buildscript {
         kotlinVersion = '1.8.0'
         dokkaVersion = '1.8.20'
         exoPlayerVersion = '1.1.1'
-        analyticsVersion = '2.0.0'
+        analyticsVersion = '2.0.1'
     }
 
     dependencies {

diff --git a/player/src/main/java/video/api/player/models/VideoOptions.kt b/player/src/main/java/video/api/player/models/VideoOptions.kt
@@ -1,7 +1,6 @@
 package video.api.player.models
 
 import video.api.player.extensions.parseAsVideoOptions
-import video.api.player.utils.Utils
 import java.net.URL
 
 /**

diff --git a/player/src/main/java/video/api/player/utils/Utils.kt b/player/src/main/java/video/api/player/utils/Utils.kt
@@ -18,7 +18,11 @@ object Utils {
     fun parseMediaUrl(
         mediaUrl: URL
     ): VideoOptions {
-        val regex = "https://[^/]+/(?>(?<type>vod|live)/)?(?>.*/)?(?<id>(vi|li)[^/^.]*).*"
+        /**
+         * Group naming is not supported before Android API 26 and crashes
+         * on very old version such as Android API 21
+         */
+        val regex = "https://[^/]+/(?>(vod|live)/)?(?>.*/)?((vi|li)[^/^.]*).*"
         val pattern = Pattern.compile(regex)
         val matcher = pattern.matcher(mediaUrl.toString())