Fixed: Numerous issues with links were reported by the Play Store.

* The Play Store highlighted the need to add "/" to the beginning of the `android:pathPattern`. As a result, we have now included "/" at the beginning of all `android:pathPattern` instances to ensure compatibility with the Play Store. * Additionally, the `android:autoVerify="true"` attribute has been added to intent filters to verify domain ownership. This was identified as an issue by both the Play Store and the App Links Assistant tool in Android Studio.
kiwix · Nov 23, 2023 · c5ee5e6 · c5ee5e6
1 parent 0dac73c
commit c5ee5e6
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 22 deletions.
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -39,6 +39,7 @@
         <action android:name="android.intent.action.VIEW" />
 
         <category android:name="android.intent.category.DEFAULT" />
+        <category android:name="android.intent.category.BROWSABLE" />
 
         <data android:scheme="content" />
         <data android:mimeType="application/octet-stream" />
@@ -52,10 +53,10 @@
 
         <data android:scheme="file" />
         <data android:mimeType="*/*" />
-        <data android:pathPattern=".*\\.zim" />
-        <data android:pathPattern=".*\\..*\\.zim" />
-        <data android:pathPattern=".*\\..*\\..*\\.zim" />
-        <data android:pathPattern=".*\\..*\\..*\\..*\\.zim" />
+        <data android:pathPattern="/.*\\.zim" />
+        <data android:pathPattern="/.*\\..*\\.zim" />
+        <data android:pathPattern="/.*\\..*\\..*\\.zim" />
+        <data android:pathPattern="/.*\\..*\\..*\\..*\\.zim" />
         <data android:host="*" />
       </intent-filter>
       <intent-filter>
@@ -66,37 +67,37 @@
 
         <data android:scheme="file" />
         <data android:mimeType="*/*" />
-        <data android:pathPattern=".*\\.zimaa" />
-        <data android:pathPattern=".*\\..*\\.zimaa" />
-        <data android:pathPattern=".*\\..*\\..*\\.zimaa" />
-        <data android:pathPattern=".*\\..*\\..*\\..*\\.zimaa" />
+        <data android:pathPattern="/.*\\.zimaa" />
+        <data android:pathPattern="/.*\\..*\\.zimaa" />
+        <data android:pathPattern="/.*\\..*\\..*\\.zimaa" />
+        <data android:pathPattern="/.*\\..*\\..*\\..*\\.zimaa" />
         <data android:host="*" />
       </intent-filter>
-      <intent-filter>
+      <intent-filter android:autoVerify="true">
         <action android:name="android.intent.action.VIEW" />
 
         <category android:name="android.intent.category.DEFAULT" />
         <category android:name="android.intent.category.BROWSABLE" />
 
         <data android:scheme="http" />
         <data android:host="*" />
-        <data android:pathPattern=".*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\..*\\.zim(aa|)" />
       </intent-filter>
-      <intent-filter>
+      <intent-filter android:autoVerify="true">
         <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" />
         <data android:host="*" />
-        <data android:pathPattern=".*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\..*\\.zim(aa|)" />
       </intent-filter>
       <intent-filter>
         <action android:name="android.intent.action.VIEW" />
@@ -106,10 +107,10 @@
 
         <data android:scheme="content" />
         <data android:host="*" />
-        <data android:pathPattern=".*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\.zim(aa|)" />
-        <data android:pathPattern=".*\\..*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\.zim(aa|)" />
+        <data android:pathPattern="/.*\\..*\\..*\\..*\\.zim(aa|)" />
       </intent-filter>
       <intent-filter>
         <action android:name="android.intent.action.VIEW" />

diff --git a/lintConfig.xml b/lintConfig.xml
@@ -47,4 +47,5 @@
   <issue id="UnusedIds" severity="warning" />
   <issue id="DataExtractionRules" severity="warning" />
   <issue id="ObsoleteSdkInt" severity="warning" />
+  <issue id="AppLinksAutoVerify" severity="warning" />
 </lint>