Proper handling of the device_id representation

miio/device.py

@@ -134,13 +134,13 @@ def do_discover(self) -> Message:
         :raises DeviceException: if the device could not be discovered."""
         m = Device.discover(self.ip)
         if m is not None:
-            self._device_id = binascii.hexlify(m.header.value.device_id)
+            self._device_id = m.header.value.device_id
             self._device_ts = m.header.value.ts
             self._discovered = True
             if self.debug > 1:
                 _LOGGER.debug(m)
             _LOGGER.debug("Discovered %s with ts: %s, token: %s",
-                          self._device_id,
+                          binascii.hexlify(self._device_id).decode(),
                           self._device_ts,
                           codecs.encode(m.checksum, 'hex'))
         else:
@@ -243,7 +243,6 @@ def send(self, command: str, parameters: Any=None, retry_count=3) -> Any:
             data, addr = s.recvfrom(1024)
             m = Message.parse(data, token=self.token)
             self._device_ts = m.header.value.ts
-            self._device_id = m.header.value.device_id
             if self.debug > 1:
                 _LOGGER.debug("recv from %s: %s", addr[0], m)
 

requirements.txt

@@ -1,7 +1,7 @@
 click
 cryptography
 pretty_cron
-construct>=2.9.30,<=2.9.31
+construct==2.9.31
 zeroconf
 attrs
 typing  # for py3.4 support

setup.py

@@ -42,7 +42,7 @@ def readme():
     packages=["miio", "mirobo"],
 
     python_requires='>=3.4',
-    install_requires=['construct>=2.9.30,<=2.9.31',
+    install_requires=['construct==2.9.31',
                       'click',
                       'cryptography',
                       'pretty_cron',