backends: bluez: handle address change after pairing

Devices with resolvable private addresses will change their address
after pairing. So for the address property to return the "correct"
value, we need to update the device's address after pairing.

Fixes: #1737

Author: dlech

CHANGELOG.rst

@@ -29,6 +29,7 @@ Changed
 
 Fixed
 -----
+* Fixed resolvable private address not updated after connecting in BlueZ backend. Fixes #1737.
 * Fixed possible ``KeyError`` when getting services in BlueZ backend. Fixes #1435.
 * Fix D-Bus connection leak when connecting to a device fails in BlueZ backend. Fixes #1698.
 * Fixed possible deadlock when connecting on WinRT backend when device is already connected. Fixes #1757.

bleak/backends/bluezdbus/client.py

@@ -284,6 +284,15 @@ async def disconnect_device() -> None:
                                     member="Pair",
                                 )
                             )
+
+                            # For resolvable private addresses, the address will
+                            # change after pairing, so we need to update that.
+                            # Hopefully there is no race condition here. D-Bus
+                            # traffic capture shows that Address change happens
+                            # at the same time as Paired property change and
+                            # that PropertiesChanged signal is sent before the
+                            # "Pair" reply is sent.
+                            self.address = manager.get_device_address(self._device_path)
                         else:
                             reply = await self._bus.call(
                                 Message(
@@ -509,6 +518,16 @@ async def pair(self, *args: Any, **kwargs: Any) -> None:
         assert reply
         assert_reply(reply)
 
+        # For resolvable private addresses, the address will
+        # change after pairing, so we need to update that.
+        # Hopefully there is no race condition here. D-Bus
+        # traffic capture shows that Address change happens
+        # at the same time as Paired property change and
+        # that PropertiesChanged signal is sent before the
+        # "Pair" reply is sent.
+        manager = await get_global_bluez_manager()
+        self.address = manager.get_device_address(self._device_path)
+
     @override
     async def unpair(self) -> None:
         """Unpair with the peripheral."""

bleak/backends/bluezdbus/manager.py

@@ -753,6 +753,21 @@ def get_device_name(self, device_path: str) -> str:
         """
         return self._get_device_property(device_path, defs.DEVICE_INTERFACE, "Name")
 
+    def get_device_address(self, device_path: str) -> str:
+        """
+        Gets the value of the "Address" property for a device.
+
+        Args:
+            device_path: The D-Bus object path of the device.
+
+        Returns:
+            The current property value.
+
+        Raises:
+            BleakError: if the device is not present in BlueZ
+        """
+        return self._get_device_property(device_path, defs.DEVICE_INTERFACE, "Address")
+
     def is_connected(self, device_path: str) -> bool:
         """
         Gets the value of the "Connected" property for a device.