Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-112164

Check ACCESS_FINE_LOCATION is no longer needed in recent Android Sdk

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: P2: Important P2: Important
    • None
    • 6.4.3
    • Connectivity: Bluetooth
    • None
    • Android

      QBluetoothDeviceDiscoveryAgentPrivate::start checks the  ACCESS_FINE_LOCATION permission and if not set, requests it. According to https://developer.android.com/guide/topics/connectivity/bluetooth/permissions the location permission is no longer needed if the app does not use scan results to derive physical location.

      I my opinion, checking for ACCESS_FINE_LOCATION should be dropped.

      See also https://bugreports.qt.io/browse/QTBUG-99590?focusedCommentId=713588 

        For Gerrit Dashboard: QTBUG-112164
        # Subject Branch Project Status CR V
        582966,7 Set sdkVersions and neverForLocation bluetooth permission assertions dev qt/qtconnectivity Status: MERGED +2 0
        582979,12 Check if location checks can be skipped with bluetooth scan dev qt/qtconnectivity Status: MERGED +2 0
        583005,4 Introduce qt_internal_add_android_permission CMake function dev qt/qtbase Status: MERGED +2 0
        583006,5 Decouple location and bluetooth permissions if API level >= 31 dev qt/qtbase Status: MERGED +2 0
        600152,1 Introduce qt_internal_add_android_permission CMake function 6.8 qt/qtbase Status: ABANDONED +1 +1
        600153,1 Decouple location and bluetooth permissions if API level >= 31 6.8 qt/qtbase Status: ABANDONED 0 0
        600156,1 Set sdkVersions and neverForLocation bluetooth permission assertions 6.8 qt/qtconnectivity Status: ABANDONED 0 0
        600157,1 Check if location checks can be skipped with bluetooth scan 6.8 qt/qtconnectivity Status: ABANDONED 0 0
        600253,3 Introduce qt_internal_add_android_permission CMake function 6.8 qt/qtbase Status: MERGED +2 +1
        600254,3 Decouple location and bluetooth permissions if API level >= 31 6.8 qt/qtbase Status: MERGED +2 0
        600263,3 Set sdkVersions and neverForLocation bluetooth permission assertions 6.8 qt/qtconnectivity Status: MERGED +2 0
        600264,3 Check if location checks can be skipped with bluetooth scan 6.8 qt/qtconnectivity Status: MERGED +2 0