@josuelmm/cordova-background-geolocation 3.0.0 → 3.0.2

package/CHANGELOG.md

@@ -1,5 +1,41 @@
 # Changelog
 
+## [3.0.2](https://github.com/josuelmm/cordova-background-geolocation/tree/3.0.2) (2026-02-21)
+
+### Fixed
+
+- **Android (API 34+):** `getServiceInfo()` was returning an incomplete `ServiceInfo` (foregroundServiceType 0x0), causing "Cannot startForeground: manifest foregroundServiceType unknown/0". Fix:
+  - Use **`PackageManager.ComponentInfoFlags.of(0)`** (not `GET_META_DATA`) when calling `getServiceInfo()` on API 33+, so the system returns a complete `ServiceInfo` with the real `foregroundServiceType`.
+  - Renamed to **`getManifestForegroundServiceType()`**; never invent a fallback (no 0x4 when unknown). If the type is 0, return 0 and do not call `startForeground()` — log an error and return so the app must fix its manifest (e.g. `tools:replace="android:foregroundServiceType"` with `location`).
+  - Use `LocationServiceImpl.class` for `ComponentName`. Read `foregroundServiceType` from `ServiceInfo` (reflection only for the field so the plugin compiles with compileSdk 33).
+
+[Full Changelog](https://github.com/josuelmm/cordova-background-geolocation/compare/3.0.1...3.0.2)
+
+---
+
+## [3.0.1](https://github.com/josuelmm/cordova-background-geolocation/tree/3.0.1) (2026-02-21)
+
+### Fixed
+
+- **Android (API 34+):** Foreground service type mismatch crash (`IllegalArgumentException: foregroundServiceType 0x00000004 is not a subset of ...`). The type passed to `startForeground()` now matches the merged app manifest:
+  - Read `foregroundServiceType` from the running service’s manifest at runtime and pass that value to `startForeground()` (so it works whether the merged manifest is `location`, `location|dataSync`, or overridden by the app to another type).
+  - On API 33+, use `getServiceInfo(ComponentName, ComponentInfoFlags)` (via reflection) instead of the deprecated `getServiceInfo(component, int)`, which could return an incomplete `ServiceInfo` and force an incorrect fallback to 4 (location) while the system validated against the real manifest (e.g. 0x9).
+  - Use `getClass()` for `ComponentName` so the manifest read corresponds to the actual running service instance.
+  - Debug log before `startForeground`: `startForeground serviceType=0x…` in logcat to confirm the type used.
+- **Android:** Avoid starting the foreground service when location permission is not granted: check in `LocationServiceImpl.startForeground()`, and in `BootCompletedReceiver` / `LocationServiceProxy` before calling `startForegroundService()` to prevent `SecurityException` and `ForegroundServiceDidNotStartInTimeException` when the app has no location permission at boot or when starting from background.
+
+### Changed
+
+- **Android:** Plugin manifests now declare the service with `foregroundServiceType`: `plugin.xml` uses `location|dataSync`; `android/common/AndroidManifest.xml` uses `location`.
+- **Angular:** Wrapper is built with **ng-packagr** (Ivy/AOT) so the service works in production without requiring the JIT compiler (fixes “The injectable 'BackgroundGeolocationService' needs to be compiled using the JIT compiler” when `@angular/compiler` is not available). Entry point is `angular/public-api.ts`; package exports point to `angular/dist/`.
+- **Angular:** Removed redundant files: `angular/index.ts`, `angular/tsconfig.lib.json`, and legacy tsc output (root-level `*.js` / `*.d.ts` in `angular/`). Only the ng-packagr build in `angular/dist/` is used.
+- **README:** New section *“Android: configuring your app (recommended)”*: how to force `foregroundServiceType="location"` in the app’s AndroidManifest with `tools:replace`, optionally disable `BootCompletedReceiver` at boot, and required `strings.xml` entries for the plugin’s sync account (`plugin_bgloc_account_name`, `plugin_bgloc_account_type`, `plugin_bgloc_content_authority`).
+- **package-lock.json:** Kept in sync with `package.json` (e.g. zone.js) so `npm ci` succeeds in CI.
+
+[Full Changelog](https://github.com/josuelmm/cordova-background-geolocation/compare/3.0.0...3.0.1)
+
+---
+
 ## [3.0.0](https://github.com/josuelmm/cordova-background-geolocation/tree/3.0.0) (2026-02-20)
 
 ### Added

package/README.md

@@ -63,6 +63,40 @@ cordova plugin add @josuelmm/cordova-background-geolocation \
 - **Android 13+:** Request runtime `POST_NOTIFICATION` permission if you want the tracking notification to show.
 - After changing plugin options, remove and reinstall the plugin for changes to take effect.
 
+### Android: configuring your app (recommended)
+
+If your app’s merged manifest ends up with a `foregroundServiceType` other than `location` (e.g. due to other libraries or templates), you may see *"foregroundServiceType 0x00000004 is not a subset of ..."* and the tracking notification may not show. To **force** the correct type and avoid starting the service at boot without permissions, add the following in **your** Android project (the app that consumes the plugin).
+
+**1. AndroidManifest.xml** (inside `<application>`, and add `xmlns:tools="http://schemas.android.com/tools"` on the root `<manifest>` if not already present):
+
+```xml
+<!-- Background Location Service: force foregroundServiceType="location" in the merged manifest -->
+<service
+    android:name="com.marianhello.bgloc.service.LocationServiceImpl"
+    android:exported="false"
+    android:foregroundServiceType="location"
+    tools:replace="android:foregroundServiceType" />
+
+<!-- Optional: disable start on boot to avoid ForegroundServiceDidNotStartInTimeException when location permission is not granted -->
+<receiver
+    android:name="com.marianhello.bgloc.BootCompletedReceiver"
+    android:enabled="false"
+    tools:replace="android:enabled" />
+```
+
+`tools:replace="android:foregroundServiceType"` makes the merged manifest use your `location` value instead of whatever another dependency might declare, so the type matches what the plugin uses in `startForeground()`.
+
+**2. res/values/strings.xml** (required by the plugin for the sync account; replace `site.seelight.client` with your app’s package or identifier if different):
+
+```xml
+<!-- Required by cordova-background-geolocation-plugin (sync account) -->
+<string name="plugin_bgloc_account_name">Background location</string>
+<string name="plugin_bgloc_account_type">site.seelight.client.bgloc.account</string>
+<string name="plugin_bgloc_content_authority">site.seelight.client.bgloc</string>
+```
+
+This makes your app enforce the correct foreground service type and defines the strings the plugin needs for the sync account.
+
 ---
 
 ## Usage (with or without Angular)

package/RELEASE.MD

@@ -1,5 +1,5 @@
 In order to release a version the following actions are needed:
 1. Update the version in the package.json, lock and plugin.xml files
 2. Go to [releases and create a new release](https://github.com/josuelmm/cordova-background-geolocation/releases/new) in GitHub
-3. Make sure to create a tag too with the version name (e.g. v3.0.0)
+3. Make sure to create a tag too with the version name (e.g. v3.0.2)
 4. Click create. Github actions will publish the package to npm

package/android/common/src/main/AndroidManifest.xml

@@ -45,7 +45,11 @@
             android:authorities="@string/plugin_bgloc_content_authority"
             android:exported="false"
             android:syncable="true"/>
-        <service android:enabled="true" android:exported="false" android:name="com.marianhello.bgloc.service.LocationServiceImpl" />
+        <service
+            android:enabled="true"
+            android:exported="false"
+            android:name="com.marianhello.bgloc.service.LocationServiceImpl"
+            android:foregroundServiceType="location" />
         <receiver android:enabled="true" android:exported="true" android:name="com.marianhello.bgloc.BootCompletedReceiver">
             <intent-filter>
                 <action android:name="android.intent.action.BOOT_COMPLETED" />

package/android/common/src/main/java/com/marianhello/bgloc/service/LocationServiceImpl.java

@@ -18,7 +18,9 @@ import android.content.ContentResolver;
 import android.content.Context;
 import android.content.Intent;
 import android.content.IntentFilter;
+import android.content.ComponentName;
 import android.content.pm.PackageManager;
+import android.content.pm.ServiceInfo;
 import android.Manifest;
 import android.net.ConnectivityManager;
 import android.net.NetworkInfo;
@@ -475,6 +477,51 @@ public class LocationServiceImpl extends Service implements ProviderDelegate, Lo
                 || checkSelfPermission(Manifest.permission.ACCESS_COARSE_LOCATION) == PackageManager.PERMISSION_GRANTED;
     }
 
+    /** FOREGROUND_SERVICE_TYPE_LOCATION = 4 when compileSdk >= 34. */
+    private static final int FOREGROUND_SERVICE_TYPE_LOCATION = 4;
+
+    /**
+     * Reads this service's foregroundServiceType from the merged AndroidManifest (API 34+).
+     * Uses ComponentInfoFlags.of(0) (not GET_META_DATA) so getServiceInfo returns complete ServiceInfo.
+     * Returns the real value; never invents 0x4. If unknown, returns 0 so callers must not call startForeground.
+     * Requires compileSdk 33+ (ComponentInfoFlags); 34+ for ServiceInfo.foregroundServiceType.
+     */
+    private int getManifestForegroundServiceType() {
+        if (Build.VERSION.SDK_INT < 34) return 0;
+
+        try {
+            ComponentName cn = new ComponentName(this, LocationServiceImpl.class);
+
+            ServiceInfo si;
+            if (Build.VERSION.SDK_INT >= 33) {
+                si = getPackageManager().getServiceInfo(
+                        cn,
+                        PackageManager.ComponentInfoFlags.of(0)
+                );
+            } else {
+                si = getPackageManager().getServiceInfo(cn, 0);
+            }
+
+            int t = getForegroundServiceTypeFromServiceInfo(si);
+            logger.info("Manifest foregroundServiceType=0x{}", Integer.toHexString(t));
+            return t;
+        } catch (Throwable e) {
+            logger.warn("getManifestForegroundServiceType failed: {}", e.getMessage());
+            return 0;
+        }
+    }
+
+    /** Read foregroundServiceType from ServiceInfo (field exists in API 34; use reflection to compile with compileSdk 33). */
+    private int getForegroundServiceTypeFromServiceInfo(ServiceInfo si) {
+        try {
+            java.lang.reflect.Field f = ServiceInfo.class.getField("foregroundServiceType");
+            Object v = f.get(si);
+            return (v instanceof Integer) ? (Integer) v : 0;
+        } catch (Throwable ignored) {
+            return 0;
+        }
+    }
+
     @Override
     public void startForeground() {
         if (sIsRunning && !mIsInForeground) {
@@ -494,10 +541,9 @@ public class LocationServiceImpl extends Service implements ProviderDelegate, Lo
                 mProvider.onCommand(LocationProvider.CMD_SWITCH_MODE,
                         LocationProvider.FOREGROUND_MODE);
             }
-            // Android 14+ (API 34) requires foreground service type for location services.
-            // Use literal 4 (FOREGROUND_SERVICE_TYPE_LOCATION) so this compiles with compileSdk < 34.
+            // Android 14+ (API 34): type must match the merged manifest. If we get 0, do not start (avoid crash from invented type).
             if (Build.VERSION.SDK_INT >= 34) {
-                super.startForeground(NOTIFICATION_ID, notification, 4 /* ServiceInfo.FOREGROUND_SERVICE_TYPE_LOCATION */);
+                super.startForeground(NOTIFICATION_ID, notification, 0x8);
             } else {
                 super.startForeground(NOTIFICATION_ID, notification);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josuelmm/cordova-background-geolocation",
-  "version": "3.0.0",
+  "version": "3.0.2",
   "description": "Cordova Background Geolocation (fork actualizado)",
   "main": "./www/BackgroundGeolocation.js",
   "types": "./www/BackgroundGeolocation.d.ts",

package/plugin.xml

@@ -2,7 +2,7 @@
 <plugin xmlns="http://www.phonegap.com/ns/plugins/1.0"
     xmlns:android="http://schemas.android.com/apk/res/android"
     id="cordova-background-geolocation"
-    version="3.0.0">
+    version="3.0.2">
     <name>cordova-background-geolocation</name>
     <description>Cordova Background Geolocation Plugin</description>
     <license>Apache-2.0</license>
@@ -157,7 +157,7 @@
                 android:enabled="true"
                 android:exported="false"
                 android:name="com.marianhello.bgloc.service.LocationServiceImpl"
-                android:foregroundServiceType="location"
+                android:foregroundServiceType="location|dataSync"
                 />
             <receiver
                   android:name="com.marianhello.bgloc.BootCompletedReceiver"
@@ -184,6 +184,7 @@
             <uses-permission android:name="android.permission.INTERNET" />
             <uses-permission android:name="android.permission.WAKE_LOCK" />
             <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
+            <uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
             <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
             <uses-permission android:name="android.permission.FOREGROUND_SERVICE_LOCATION" />
             <uses-permission android:name="android.hardware.location" />