expo-location 18.0.10-canary-20250331-817737a → 18.0.10

package/CHANGELOG.md

@@ -8,47 +8,54 @@
 
 ### 🐛 Bug fixes
 
-- [iOS] Remove restarting all services when CLLocationManager reports an error ([#35478](https://github.com/expo/expo/pull/35478) by [@chrfalch](https://github.com/chrfalch))
-- [Android] Add missing ProGuard rule to fix task consumer failed ([#34098](https://github.com/expo/expo/pull/34098) by [@cornejobarraza](https://github.com/cornejobarraza))
-- [iOS] `startLocationUpdatesAsync` should not require background permissions ([#33617](https://github.com/expo/expo/pull/33617) by [@andrejpavlovic](https://github.com/andrejpavlovic)
-
 ### 💡 Others
 
-- On Android, remove dependency on `smart-location-lib`. ([#33609](https://github.com/expo/expo/pull/33609) by [@alanjhughes](https://github.com/alanjhughes))
-- [Android] Started using expo modules gradle plugin. ([#34176](https://github.com/expo/expo/pull/34176) by [@lukmccall](https://github.com/lukmccall))
+## 18.0.10 — 2025-04-01
 
-## 18.0.9 - 2025-03-31
+### 🐛 Bug fixes
+
+- [iOS] Fixed issue with some permission request flows resolving too soon on iOS. ([#35693](https://github.com/expo/expo/pull/35693) by [@chrfalch](https://github.com/chrfalch))
+- [iOS] Remove restarting all services when CLLocationManager reports an error ([#35478](https://github.com/expo/expo/pull/35478) by [@chrfalch](https://github.com/chrfalch))
+
+## 18.0.9 — 2025-03-31
 
 _This version does not introduce any user-facing changes._
 
-## 18.0.8 - 2025-03-14
+## 18.0.8 — 2025-03-14
 
 ### 💡 Others
 
 - On iOS, added setting the scope value as per our documentation. ([#35452](https://github.com/expo/expo/pull/35452) by [@chrfalch](https://github.com/chrfalch))
 
+## 18.0.7 — 2025-02-19
+
+- On Android, remove dependency on `smart-location-lib`. ([#33609](https://github.com/expo/expo/pull/33609) by [@alanjhughes](https://github.com/alanjhughes))
+- [Android] Started using expo modules gradle plugin. ([#34176](https://github.com/expo/expo/pull/34176) by [@lukmccall](https://github.com/lukmccall))
+
 ## 18.0.7 - 2025-02-19
 
+- ([iOS][location] Add scope to foreground and background iOS permissions as pr. documentation (#35452))
+
 ### 🐛 Bug fixes
 
 - [iOS] Added guards to avoid task options to crash the app. ([#35477](https://github.com/expo/expo/pull/35477) by [@chrfalch](https://github.com/chrfalch))
 - [iOS] Added error handler to the streaming location/heading methods since these can fail while streaming ([#35004](https://github.com/expo/expo/pull/35004) by [@chrfalch](https://github.com/chrfalch))
 
-## 18.0.6 - 2025-02-10
+## 18.0.6 — 2025-02-10
 
 ### 🐛 Bug fixes
 
 - [Android] Use less specific exception in catch block of `resolveUserSettingsForRequest`. ([#34784](https://github.com/expo/expo/pull/34784) by [@alanjhughes](https://github.com/alanjhughes))
 
-## 18.0.5 - 2025-01-10
+## 18.0.5 — 2025-01-10
 
 _This version does not introduce any user-facing changes._
 
-## 18.0.4 - 2024-12-10
+## 18.0.4 — 2024-12-10
 
 _This version does not introduce any user-facing changes._
 
-## 18.0.3 - 2024-11-29
+## 18.0.3 — 2024-11-29
 
 _This version does not introduce any user-facing changes._
 

package/android/build.gradle

@@ -1,20 +1,24 @@
-plugins {
-  id 'com.android.library'
-  id 'expo-module-gradle-plugin'
-}
+apply plugin: 'com.android.library'
 
 group = 'host.exp.exponent'
-version = '18.0.2'
+version = '18.0.10'
+
+def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
+apply from: expoModulesCorePlugin
+applyKotlinExpoModulesCorePlugin()
+useCoreDependencies()
+useDefaultAndroidSdkVersions()
+useExpoPublishing()
 
 android {
   namespace "expo.modules.location"
   defaultConfig {
     versionCode 29
-    versionName "18.0.2"
-    consumerProguardFiles("proguard-rules.pro")
+    versionName "18.0.10"
   }
 }
 
 dependencies {
   api 'com.google.android.gms:play-services-location:21.0.1'
+  implementation project(":${project.name}\$io.nlopez.smartlocation-jetified-aar")
 }

package/android/src/main/java/expo/modules/location/LocationHelpers.kt

@@ -16,6 +16,8 @@ import expo.modules.location.records.LocationLastKnownOptions
 import expo.modules.location.records.LocationOptions
 import expo.modules.location.records.LocationResponse
 import expo.modules.location.records.PermissionRequestResponse
+import io.nlopez.smartlocation.location.config.LocationAccuracy
+import io.nlopez.smartlocation.location.config.LocationParams
 import kotlin.coroutines.resume
 import kotlin.coroutines.resumeWithException
 import kotlin.coroutines.suspendCoroutine
@@ -108,16 +110,16 @@ class LocationHelpers {
 
     private fun mapOptionsToLocationParams(options: LocationOptions): LocationParams {
       val accuracy = options.accuracy
-      val locationParams = buildLocationParamsForAccuracy(accuracy)
+      val locationParamsBuilder = buildLocationParamsForAccuracy(accuracy)
 
       options.timeInterval?.let {
-        locationParams.interval = it
+        locationParamsBuilder.setInterval(it)
       }
       options.distanceInterval?.let {
-        locationParams.distance = it.toFloat()
+        locationParamsBuilder.setDistance(it.toFloat())
       }
 
-      return locationParams
+      return locationParamsBuilder.build()
     }
 
     private fun mapAccuracyToPriority(accuracy: Int): Int {
@@ -129,15 +131,42 @@ class LocationHelpers {
       }
     }
 
-    private fun buildLocationParamsForAccuracy(accuracy: Int): LocationParams {
+    private fun buildLocationParamsForAccuracy(accuracy: Int): LocationParams.Builder {
       return when (accuracy) {
-        LocationModule.ACCURACY_LOWEST -> LocationParams(accuracy = LocationAccuracy.LOWEST, distance = 3000f, interval = 10000)
-        LocationModule.ACCURACY_LOW -> LocationParams(accuracy = LocationAccuracy.LOW, distance = 1000f, interval = 5000)
-        LocationModule.ACCURACY_BALANCED -> LocationParams(accuracy = LocationAccuracy.MEDIUM, distance = 100f, interval = 3000)
-        LocationModule.ACCURACY_HIGH -> LocationParams(accuracy = LocationAccuracy.HIGH, distance = 50f, interval = 2000)
-        LocationModule.ACCURACY_HIGHEST -> LocationParams(accuracy = LocationAccuracy.HIGH, distance = 25f, interval = 1000)
-        LocationModule.ACCURACY_BEST_FOR_NAVIGATION -> LocationParams(accuracy = LocationAccuracy.HIGH, distance = 0f, interval = 500)
-        else -> LocationParams(accuracy = LocationAccuracy.MEDIUM, distance = 100f, interval = 3000)
+        LocationModule.ACCURACY_LOWEST -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.LOWEST)
+          .setDistance(3000f)
+          .setInterval(10000)
+
+        LocationModule.ACCURACY_LOW -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.LOW)
+          .setDistance(1000f)
+          .setInterval(5000)
+
+        LocationModule.ACCURACY_BALANCED -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.MEDIUM)
+          .setDistance(100f)
+          .setInterval(3000)
+
+        LocationModule.ACCURACY_HIGH -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.HIGH)
+          .setDistance(50f)
+          .setInterval(2000)
+
+        LocationModule.ACCURACY_HIGHEST -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.HIGH)
+          .setDistance(25f)
+          .setInterval(1000)
+
+        LocationModule.ACCURACY_BEST_FOR_NAVIGATION -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.HIGH)
+          .setDistance(0f)
+          .setInterval(500)
+
+        else -> LocationParams.Builder()
+          .setAccuracy(LocationAccuracy.MEDIUM)
+          .setDistance(100f)
+          .setInterval(3000)
       }
     }
 

package/android/src/main/java/expo/modules/location/LocationModule.kt

@@ -4,7 +4,6 @@ import android.Manifest
 import android.app.Activity
 import android.content.Context
 import android.content.Intent
-import android.content.pm.PackageManager
 import android.hardware.GeomagneticField
 import android.hardware.Sensor
 import android.hardware.SensorEvent
@@ -12,14 +11,11 @@ import android.hardware.SensorEventListener
 import android.hardware.SensorManager
 import android.location.Geocoder
 import android.location.Location
-import android.location.LocationManager
 import android.os.Build
 import android.os.Bundle
 import android.os.Looper
 import android.util.Log
 import androidx.annotation.ChecksSdkIntAtLeast
-import androidx.core.app.ActivityCompat
-import androidx.core.location.LocationManagerCompat
 import androidx.core.os.bundleOf
 import com.google.android.gms.common.api.ApiException
 import com.google.android.gms.common.api.CommonStatusCodes
@@ -55,6 +51,9 @@ import expo.modules.location.records.ReverseGeocodeLocation
 import expo.modules.location.records.ReverseGeocodeResponse
 import expo.modules.location.taskConsumers.GeofencingTaskConsumer
 import expo.modules.location.taskConsumers.LocationTaskConsumer
+import io.nlopez.smartlocation.SmartLocation
+import io.nlopez.smartlocation.geocoding.utils.LocationAddress
+import io.nlopez.smartlocation.location.config.LocationParams
 import java.util.Locale
 import kotlin.coroutines.resume
 import kotlin.coroutines.resumeWithException
@@ -158,12 +157,21 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
     }
 
     AsyncFunction<LocationProviderStatus>("getProviderStatusAsync") {
-      return@AsyncFunction getProviderStatus()
+      val state = SmartLocation.with(mContext).location().state()
+
+      return@AsyncFunction LocationProviderStatus().apply {
+        backgroundModeEnabled = state.locationServicesEnabled()
+        gpsAvailable = state.isGpsAvailable
+        networkAvailable = state.isNetworkAvailable
+        locationServicesEnabled = state.locationServicesEnabled()
+        passiveAvailable = state.isPassiveAvailable
+      }
     }
 
     AsyncFunction("watchDeviceHeading") { watchId: Int ->
       mHeadingId = watchId
-      return@AsyncFunction startHeadingUpdate()
+      startHeadingUpdate()
+      return@AsyncFunction
     }
 
     AsyncFunction("watchPositionImplAsync") { watchId: Int, options: LocationOptions, promise: Promise ->
@@ -206,6 +214,7 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
       } else {
         removeLocationUpdatesForRequest(watchId)
       }
+      return@AsyncFunction
     }
 
     AsyncFunction("geocodeAsync") Coroutine { address: String ->
@@ -332,23 +341,6 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
     } ?: throw NoPermissionsModuleException()
   }
 
-  private fun getProviderStatus(): LocationProviderStatus {
-    val manager = mContext.getSystemService(Context.LOCATION_SERVICE) as LocationManager
-
-    val isGpsAvailable = manager.isProviderEnabled(LocationManager.GPS_PROVIDER)
-    val isNetworkAvailable = manager.isProviderEnabled(LocationManager.NETWORK_PROVIDER)
-    val isLocationServicesEnabled = LocationManagerCompat.isLocationEnabled(manager)
-    val isPassiveAvailable = manager.isProviderEnabled(LocationManager.PASSIVE_PROVIDER)
-
-    return LocationProviderStatus().apply {
-      backgroundModeEnabled = isLocationServicesEnabled
-      gpsAvailable = isGpsAvailable
-      networkAvailable = isNetworkAvailable
-      locationServicesEnabled = isLocationServicesEnabled
-      passiveAvailable = isPassiveAvailable
-    }
-  }
-
   private suspend fun requestBackgroundPermissionsAsync(): PermissionRequestResponse {
     if (!isBackgroundPermissionInManifest()) {
       throw NoPermissionInManifestException("ACCESS_BACKGROUND_LOCATION")
@@ -508,54 +500,27 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
   }
 
   private fun startHeadingUpdate() {
-    val locationManager = mContext.getSystemService(Context.LOCATION_SERVICE) as LocationManager
-    if (ActivityCompat.checkSelfPermission(mContext, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED &&
-      ActivityCompat.checkSelfPermission(mContext, Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED
-    ) {
-      return
-    }
-    val lastLocation =
-      locationManager.getLastKnownLocation(LocationManager.GPS_PROVIDER)
-        ?: locationManager.getLastKnownLocation(LocationManager.NETWORK_PROVIDER)
-
-    if (lastLocation != null) {
+    val locationControl = SmartLocation.with(mContext).location().oneFix().config(LocationParams.BEST_EFFORT)
+    val currLoc = locationControl.lastLocation
+    if (currLoc != null) {
       mGeofield = GeomagneticField(
-        lastLocation.latitude.toFloat(),
-        lastLocation.longitude.toFloat(),
-        lastLocation.altitude.toFloat(),
+        currLoc.latitude.toFloat(), currLoc.longitude.toFloat(), currLoc.altitude.toFloat(),
         System.currentTimeMillis()
       )
     } else {
-      val locationRequest = LocationRequest.Builder(
-        LocationRequest.PRIORITY_HIGH_ACCURACY,
-        0L
-      ).setMaxUpdates(1)
-        .build()
-
-      val locationCallback = object : LocationCallback() {
-        override fun onLocationResult(locationResult: LocationResult) {
-          locationResult.lastLocation?.let {
-            mGeofield = GeomagneticField(
-              it.latitude.toFloat(),
-              it.longitude.toFloat(),
-              it.altitude.toFloat(),
-              System.currentTimeMillis()
-            )
-          }
-        }
+      locationControl.start { location: Location ->
+        mGeofield = GeomagneticField(
+          location.latitude.toFloat(), location.longitude.toFloat(), location.altitude.toFloat(),
+          System.currentTimeMillis()
+        )
       }
-      mLocationProvider.requestLocationUpdates(locationRequest, locationCallback, Looper.getMainLooper())
     }
     mSensorManager.registerListener(
       this,
       mSensorManager.getDefaultSensor(Sensor.TYPE_MAGNETIC_FIELD),
       SensorManager.SENSOR_DELAY_NORMAL
     )
-    mSensorManager.registerListener(
-      this,
-      mSensorManager.getDefaultSensor(Sensor.TYPE_ACCELEROMETER),
-      SensorManager.SENSOR_DELAY_NORMAL
-    )
+    mSensorManager.registerListener(this, mSensorManager.getDefaultSensor(Sensor.TYPE_ACCELEROMETER), SensorManager.SENSOR_DELAY_NORMAL)
   }
 
   private fun sendUpdate() {
@@ -633,6 +598,7 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
   private fun stopWatching() {
     // if permissions not granted it won't work anyway, but this can be invoked when permission dialog appears
     if (Geocoder.isPresent() && !isMissingForegroundPermissions()) {
+      SmartLocation.with(mContext).geocoding().stop()
       mGeocoderPaused = true
     }
     for (requestId in mLocationCallbacks.keys) {
@@ -699,10 +665,8 @@ class LocationModule : Module(), LifecycleEventListener, SensorEventListener, Ac
       locations?.let { location ->
         location.let {
           val results = it.mapNotNull { address ->
-            val newLocation = Location(LocationManager.GPS_PROVIDER)
-            newLocation.latitude = address.latitude
-            newLocation.longitude = address.longitude
-            GeocodeResponse.from(newLocation)
+            val locationAddress = LocationAddress(address)
+            GeocodeResponse.from(locationAddress.location)
           }
           continuation.resume(results)
         }

package/build/LocationEventEmitter.web.d.ts

@@ -1,2 +1,2 @@
-export declare const LocationEventEmitter: import("expo").EventEmitterType<Record<never, never>>;
+export declare const LocationEventEmitter: import("expo-modules-core/types").EventEmitter<Record<never, never>>;
 //# sourceMappingURL=LocationEventEmitter.web.d.ts.map

package/build/LocationEventEmitter.web.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"LocationEventEmitter.web.d.ts","sourceRoot":"","sources":["../src/LocationEventEmitter.web.ts"],"names":[],"mappings":"AAEA,eAAO,MAAM,oBAAoB,uDAAqB,CAAC"}
+{"version":3,"file":"LocationEventEmitter.web.d.ts","sourceRoot":"","sources":["../src/LocationEventEmitter.web.ts"],"names":[],"mappings":"AAEA,eAAO,MAAM,oBAAoB,sEAAqB,CAAC"}

package/expo-module.config.json

@@ -1,9 +1,22 @@
 {
-  "platforms": ["apple", "android"],
+  "platforms": [
+    "apple",
+    "android"
+  ],
   "apple": {
-    "modules": ["LocationModule"]
+    "modules": [
+      "LocationModule"
+    ]
   },
   "android": {
-    "modules": ["expo.modules.location.LocationModule"]
+    "modules": [
+      "expo.modules.location.LocationModule"
+    ],
+    "gradleAarProjects": [
+      {
+        "name": "io.nlopez.smartlocation-jetified-aar",
+        "aarFilePath": "android/libs/io.nlopez.smartlocation-3.3.3-jetified.aar"
+      }
+    ]
   }
 }

package/ios/LocationModule.swift

@@ -163,13 +163,9 @@ public final class LocationModule: Module {
     // Background location
 
     AsyncFunction("startLocationUpdatesAsync") { (taskName: String, options: [String: Any]) in
-      // There are two ways of starting this service.
-      // 1. As a background location service, this requires the background location permission.
-      // 2. As a user-initiated foreground service, this does NOT require the background location permission.
-      // Unfortunately, we cannot distinguish between those cases.
-      // So we only check foreground permission which needs to be granted in both cases.
       try ensureLocationServicesEnabled()
       try ensureForegroundLocationPermissions(appContext)
+      try ensureBackgroundLocationPermissions(appContext)
 
       guard CLLocationManager.significantLocationChangeMonitoringAvailable() else {
         throw Exceptions.LocationUpdatesUnavailable()

package/ios/Requesters/EXBackgroundLocationPermissionRequester.m

@@ -10,6 +10,7 @@ static SEL alwaysAuthorizationSelector;
 @interface EXBackgroundLocationPermissionRequester ()
 
 @property (nonatomic, assign) bool wasAsked;
+@property (nonatomic, assign) bool isWaitingForTimeout;
 
 @end
 
@@ -19,6 +20,7 @@ static SEL alwaysAuthorizationSelector;
 {
   if (self = [super init]) {
     _wasAsked = false;
+    _isWaitingForTimeout = false;
   }
   return self;
 }
@@ -37,10 +39,28 @@ static SEL alwaysAuthorizationSelector;
 {
   if ([EXBaseLocationRequester isConfiguredForAlwaysAuthorization] && [self.locationManager respondsToSelector:alwaysAuthorizationSelector]) {
     _wasAsked = true;
-    [[NSNotificationCenter defaultCenter] addObserver:self
-                                             selector:@selector(handleAppBecomingActive)
-                                                 name:UIApplicationDidBecomeActiveNotification
-                                               object:nil];
+    CLAuthorizationStatus status = [self.locationManager authorizationStatus];
+    
+    if (status == kCLAuthorizationStatusAuthorizedWhenInUse) {
+      // We already have a foreground permission granted:
+      // When asking for background location, we might or might not have asked for foreground permission
+      // before we get here. An issue here is if the user has a temporary permission ("Allow once") - which
+      // results in the status being "kCLAuthorizationStatusAuthorizedWhenInUse" - without us knowing.
+      // We need to handle this special case which is not possible to detect through the API.
+      // What we do is that we'll wait 1.5 seconds on an UIApplicationWillResignActiveNotification
+      // notification (which will be emitted almost directly if the permission dialog is displayed). If the permission
+      // dialog is not displayed we'll timeout and can resolve the waiting promise with an updated denied status.
+      [[NSNotificationCenter defaultCenter] addObserver:self
+                                               selector:@selector(handleAppBecomingInactive)
+                                                   name:UIApplicationWillResignActiveNotification
+                                                 object:nil];
+      
+      // Setup timeout - if no permission dialog was displayed we can just stop listening and deny
+      // the request
+      [self setupAppInactivateTimeout];
+    }
+    
+    // Request permissions
     ((void (*)(id, SEL))objc_msgSend)(self.locationManager, alwaysAuthorizationSelector);
   } else {
     self.reject(@"ERR_LOCATION_INFO_PLIST", @"One of the `NSLocation*UsageDescription` keys must be present in Info.plist to be able to use geolocation.", nil);
@@ -50,9 +70,6 @@ static SEL alwaysAuthorizationSelector;
   }
 }
 
-// If user selects "Keep Only While Using" option, the `locationManagerDidChangeAuthorization` won't be called.
-// So we don't know when we should resolve promise.
-// Hovewer, we can check for `UIApplicationDidBecomeActiveNotification` event which is called when permissions modal disappears.
 - (void)handleAppBecomingActive
 {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
@@ -63,6 +80,47 @@ static SEL alwaysAuthorizationSelector;
   }
 }
 
+- (void)handleAppBecomingInactive
+{
+  // Let's wait until the app becomes inactive - this happens when OS displays the
+  // permission dialog - then we can cancel the timeout handler.
+
+  _isWaitingForTimeout = false;
+  [[NSNotificationCenter defaultCenter] removeObserver:self];
+  
+  // When the app is inactive it means that a permission dialog is showing and we should ask to be
+  // notified when the dialog is closed:
+  [[NSNotificationCenter defaultCenter] addObserver:self
+                                           selector:@selector(handleAppBecomingActive)
+                                               name:UIApplicationDidBecomeActiveNotification
+                                             object:nil];
+}
+
+- (void)setupAppInactivateTimeout
+{
+  _isWaitingForTimeout = true;
+  
+  // Obtain a reference to the current queue
+  dispatch_queue_t currentQueue = dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_BACKGROUND, 0);
+
+  // Calculate the time for the delay
+  dispatch_time_t delayTime = dispatch_time(DISPATCH_TIME_NOW, (int64_t)(1.5 * NSEC_PER_SEC));
+
+  EX_WEAKIFY(self);
+  
+  // Schedule the block to be executed after the delay
+  dispatch_after(delayTime, currentQueue, ^{
+    EX_ENSURE_STRONGIFY(self)
+    // Check if we are still waiting - ie. we haven't seen a permission dialog
+    if (self.isWaitingForTimeout && self.resolve) {
+      self.isWaitingForTimeout = false;
+      self.resolve([self getPermissions]);
+      self.resolve = nil;
+      self.reject = nil;
+    }
+  });
+}
+
 - (NSDictionary *)parsePermissions:(CLAuthorizationStatus)systemStatus
 {
   EXPermissionStatus status;

package/ios/Requesters/EXBaseLocationRequester.h

@@ -3,8 +3,9 @@
 #import <CoreLocation/CLLocationManager.h>
 
 #import <ExpoModulesCore/EXPermissionsInterface.h>
+#import <CoreLocation/CLLocationManagerDelegate.h>
 
-@interface EXBaseLocationRequester : NSObject<EXPermissionsRequester>
+@interface EXBaseLocationRequester : NSObject<EXPermissionsRequester, CLLocationManagerDelegate>
 
 @property (nonatomic, strong) CLLocationManager *locationManager;
 @property (nonatomic, strong) EXPromiseResolveBlock resolve;
@@ -16,6 +17,4 @@
 - (void)requestLocationPermissions;
 - (NSDictionary *)parsePermissions:(CLAuthorizationStatus)systemStatus;
 
-- (void)locationManager:(CLLocationManager *)manager didChangeAuthorizationStatus:(CLAuthorizationStatus)status;
-
 @end

package/ios/Requesters/EXBaseLocationRequester.m

@@ -4,12 +4,11 @@
 #import <ExpoModulesCore/EXUtilities.h>
 
 #import <objc/message.h>
-#import <CoreLocation/CLLocationManagerDelegate.h>
 
 @interface EXBaseLocationRequester () <CLLocationManagerDelegate>
 
 @property (nonatomic, assign) bool locationManagerWasCalled;
-
+@property (nonatomic, assign) CLAuthorizationStatus beginStatus;
 
 @end
 
@@ -73,25 +72,47 @@
     //    make Expo developers receive this kind of messages nor add our own default usage description,
     //    we try to fool the static analyzer and construct the selector in runtime.
     //    This way behavior of this requester is governed by provided NSLocationUsageDescriptions.
+    
+    // 2. Location permission request types
     //
-    // 2. Why there's no way to call specifically whenInUse or always authorization?
+    //    Foreground
+    //    - "Allow once"
+    //    - "Allow while using App"
+    //    - "Don't allow"
     //
-    //    The requester sets itself as the delegate of the CLLocationManager, so when the user responds
-    //    to a permission requesting dialog, manager calls `locationManager:didChangeAuthorizationStatus:` method.
-    //    To be precise, manager calls this method in two circumstances:
-    //      - right when `request*Authorization` method is called,
-    //      - when `authorizationStatus` changes.
-    //    With this behavior we aren't able to support the following use case:
-    //      - app requests `whenInUse` authorization
-    //      - user allows `whenInUse` authorization
-    //      - `authorizationStatus` changes from `undetermined` to `whenInUse`, callback is called, promise is resolved
-    //      - app wants to escalate authorization to `always`
-    //      - user selects `whenInUse` authorization (iOS 11+)
-    //      - `authorizationStatus` doesn't change, so callback is not called and requester can't know whether
-    //        user responded to the dialog selecting `whenInUse` or is still deciding
-    //    To support this use case we will have to change the way location authorization is requested
-    //    from promise-based to listener-based.
-
+    //    Background
+    //    - "Keep only while using"
+    //    - "Change to always allow"
+    //
+    // Requesting background permissions directly without first asking for foreground permissions is the
+    // same as asking for foreground permissions and then asking for background permissions.
+    //
+    // "Allow once" is a temporary permission (limited to the current app session). It is not possible to get
+    // info from the API about wether or not the current permission is temporary. You cannot request background
+    // permissions with a temporary token - a background request will then return denied.
+    //
+    // Requesting background permissions directly and "Allow while using the App" gives you a provisional
+    // background permission that can later be elevated to a full "Always allow" permission.
+    // You will be asked at a later point if you want to convert to "Always allow". The system waits until
+    // you have started using the newly aquired permission before showing the permission dialog.
+    //
+    // Test the following scenarios in BareExpo -> APIs -> Location
+    // ------------------------------------------------------------
+    // (before tests, make sure to clear any location permissions and restart the app)
+    //
+    //   rfp = requestForegroundPermissionsAsync, fp: Actual foreground permission given
+    //   rbp = requestBackgroundPermissionsAsync, bg: Actual background permission given
+    //
+    // - rfp -> "Allow once", then rbp -> no dialog                             = (fp: granted (temporary), bg: denied after 1.5 seconds)
+    // - rfp -> "Allow while using App", then rbp -> "Keep only while using"    = (fp: granted, bg: denied)
+    // - rfp -> "Allow while using App", then rbp -> "Change to always allow"   = (fp: granted, bg: granted)
+    // - rfp -> "Don't allow", then rbp -> no dialog                            = (fp: denied, bg: denied)
+    // - rbp -> "Allow once", no more dalogs                                    = (fp: granted (temporary), bg: denied)
+    // - rbp -> "Allow while using App", no more dialogs                        = (fp: granted, bg: granted (provisional))
+    // - rbp -> "Don't allow"                                                   = (fp: denied, bg: denied)
+    
+    // Save start statue and call requestLocationPermissions
+    _beginStatus = [self.locationManager authorizationStatus];
     [self requestLocationPermissions];
   }
 }
@@ -107,38 +128,15 @@
   }
 }
 
-- (void)locationManager:(CLLocationManager *)manager didChangeAuthorizationStatus:(CLAuthorizationStatus)status
-{
-  // TODO: Permissions.LOCATION issue (search by this phrase)
-  // if Permissions.LOCATION is being called for the first time on iOS devide and prompts for user action it might not call this callback at all
-  // it happens if user requests more that one permission at the same time via Permissions.askAsync(...) and LOCATION dialog is not being called first
-  // to reproduce this find NCL code testing that
-  if (status == kCLAuthorizationStatusNotDetermined || !_locationManagerWasCalled) {
-    // CLLocationManager calls this delegate method once on start with kCLAuthorizationNotDetermined even before the user responds
-    // to the "Don't Allow" / "Allow" dialog box. This isn't the event we care about so we skip it. See:
-    // http://stackoverflow.com/questions/30106341/swift-locationmanager-didchangeauthorizationstatus-always-called/30107511#30107511
-    _locationManagerWasCalled = true;
-    return;
-  }
-
-  if (_resolve) {
-    _resolve([self getPermissions]);
-    _resolve = nil;
-    _reject = nil;
-  }
-}
-
 - (void)locationManagerDidChangeAuthorization:(CLLocationManager *)manager
 {
-  CLAuthorizationStatus status = [manager authorizationStatus];
-  if (status == kCLAuthorizationStatusNotDetermined || !_locationManagerWasCalled) {
+  CLAuthorizationStatus nextState = [manager authorizationStatus];
+  if (_beginStatus == nextState && !_locationManagerWasCalled) {
     // CLLocationManager calls this delegate method once on start with kCLAuthorizationNotDetermined even before the user responds
     // to the "Don't Allow" / "Allow" dialog box. This isn't the event we care about so we skip it. See:
     // http://stackoverflow.com/questions/30106341/swift-locationmanager-didchangeauthorizationstatus-always-called/30107511#30107511
     _locationManagerWasCalled = true;
-    if (status != kCLAuthorizationStatusAuthorizedWhenInUse) {
-      return;
-    }
+    return;
   }
 
   if (_resolve) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-location",
-  "version": "18.0.10-canary-20250331-817737a",
+  "version": "18.0.10",
   "description": "Allows reading geolocation information from the device. Your app can poll for the current location or subscribe to location update events.",
   "main": "build/index.js",
   "types": "build/index.d.ts",
@@ -39,9 +39,10 @@
     "preset": "expo-module-scripts"
   },
   "devDependencies": {
-    "expo-module-scripts": "4.0.5-canary-20250331-817737a"
+    "expo-module-scripts": "^4.0.4"
   },
   "peerDependencies": {
-    "expo": "53.0.0-canary-20250331-817737a"
-  }
+    "expo": "*"
+  },
+  "gitHead": "b08a0bd52965f85871c12c31da16a45e2cd26c4c"
 }

package/android/proguard-rules.pro

@@ -1,3 +0,0 @@
-# https://github.com/expo/expo/issues/3918
-
--keep class expo.modules.location.taskConsumers.** { *; }

package/android/src/main/java/expo/modules/location/LocationParams.kt

@@ -1,14 +0,0 @@
-package expo.modules.location
-
-data class LocationParams(
-  val accuracy: LocationAccuracy,
-  var distance: Float,
-  var interval: Long
-)
-
-enum class LocationAccuracy {
-  LOWEST,
-  LOW,
-  MEDIUM,
-  HIGH
-}