Not working with newest firmware version 3.3.9_003073

[ultrara1n]

Using this great code till today, after Xiaomi had to change the way of communication.
Getting the following result when just trying to get info data from vacuum:

ERROR:mirobo.vacuum:got error when receiving: timed out
Traceback (most recent call last):
File "/usr/local/bin/mirobo", line 11, in
sys.exit(cli())
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 1043, in invoke
return Command.invoke(self, ctx)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.4/dist-packages/click/decorators.py", line 17, in new_func
return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python3.4/dist-packages/mirobo/cli.py", line 46, in cli
ctx.invoke(status)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.4/dist-packages/click/decorators.py", line 64, in new_func
return ctx.invoke(f, obj, *args[1:], **kwargs)
File "/usr/local/lib/python3.4/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.4/dist-packages/mirobo/cli.py", line 59, in status
res = vac.status()
File "/usr/local/lib/python3.4/dist-packages/mirobo/vacuum.py", line 150, in status
return VacuumStatus(self.send("get_status")[0])
File "/usr/local/lib/python3.4/dist-packages/mirobo/vacuum.py", line 121, in send
data, addr = s.recvfrom(1024)
socket.timeout: timed out

[jcastro]

umm it's working for me! maybe the token changed or something else?

[ultrara1n]

Have you updated your vacuum to the latest Firmware version? According to http://www.roboter-forum.com/showthread.php?18995-Neues-Firmware-Update!&p=254656&viewfull=1#post254656 there was a change for the sent packages to the robot. They are now signed with a timestamp to dismiss too old packages. The token haven't changed, because the App of the User from the linked Post ist already working, after updating to the newest Version.

[SchumyHao]

@rytilahti I think you can take a look my pull request. I fixed the timestamp issue

[jcastro]

@ultrara1n yeah I did (see image below). Aside from these errors, which I was getting them before the update, everything seems to work as usual

2017-06-07 10:17:28 ERROR (<concurrent.futures.thread.ThreadPoolExecutor object at 0x2ba90ccbed30>_5) [mirobo.vacuum] got error when receiving: timed out 2017-06-07 10:17:28 ERROR (<concurrent.futures.thread.ThreadPoolExecutor object at 0x2ba90ccbed30>_5) [custom_components.switch.xiaomi_vacuum] Got exception while fetching the state: timed out

[rytilahti]

@jcastro maybe your clock on the vacuum is also in correct time? Currently we are sending the current timestamp out, which could explain why it is working.

@SchumyHao do you mind rebasing that change on the current HEAD, or should I just incorporate "replaying" the device timestamp + some short delay?

[jcastro]

@rytilahti umm just checked it and it seems fine! Also checked my docker installation and it's also correct

root@unBeast:/usr/src/app# date
Wed Jun  7 12:14:25 CEST 2017
root@unBeast:/usr/src/app# 

[rytilahti]

@jcastro yes, but does the robot return something similar? (when doing mirobo -d or -dd) Mine (which has no internet access) reports: ts = 1970-01-01 15:55:39

[SchumyHao]

@rytilahti I'll create a new pull request later.
Because after you refactoring the code, my early pull request has merge conflict.

[rytilahti]

@SchumyHao ok, thanks! Please also take a note on my commentary on the code parts. I think it should be enough just to save the timestamp returned in responses and increment it with timedelta for a bit.

[jcastro]

@rytilahti yep!

value = Container: 
            ts = 2017-06-07 12:31:32

[chrostek]

any news here? my mirobo still does not work with this script with firmware 3.3.9 (worked with older one).
the patch from rytilahti also does not work for me. with the original script the
ts is correct (localtime), with the patch from rytilahti the ts is+2h (utc+4, localtime+2) my localtime is "Europe/Berlin (CEST, +0200)"

[chrostek]

iobroker-community-adapters/ioBroker.mihome-vacuum@2e28df0

it seems that there was a protocol change with 3.3.9

[chrostek]

@ultrara1n can you try to discover the token via one of this ways: https://github.com/ioBroker/ioBroker.mihome-vacuum#konfiguration
in my case the token discovered via the python script was wrong (i have done a complete reset and discovered new after the firmware update). the token in the sqlite database (iOS) was another one. with the other one the communication works again ...