-
Notifications
You must be signed in to change notification settings - Fork 104
sleekxmpp.basexmpp ERROR day is out of range for month #45
Comments
Look at my post here: home-assistant/core#16186 Can be fixed, but perhaps you will run in to other issues later, like I did. |
More to the point, it seems that this issue is upstream in sleekXMPP: fritzy/SleekXMPP#478 Considering how long that issue has been open, we may need to figure out a workaround here. |
Any updates on this one? I got the same error and wonder if there is a fix for it. |
Also finding the same error |
Same issue of the OP in Home Assistant 0.81.2 |
Same issue here running hassio 0.81.6 |
If I fix the date out of range error via fritzy/SleekXMPP#478 (comment), I get this error instead:
|
@kmlucy Line 111 Replaced with...
Line 115 Replaced with...
|
@alagarath I actually ended up just reverting to the previous dependencies, which fixed the problem for the moment. It's not a long term solution, but it fixes the issue for the moment: home-assistant/core#18054 (comment) |
@kmlucy |
Would be nice if the working (latest) dependencies will be reported in this topic by the first one who see's it working. |
Seeing this on 0.82.1 |
Hey, did you managed to get it working ? |
The good news is that it appears that @fritzy is active again on SleekXMPP as two commits were put in today. I just left a note on the open issue over there with a hopeful prod. If we can't get a new version of SleekXMPP then we can simply pin sucks to use 1.3.2 to fix this issue. |
@marlonalkan I ran the following in my docker container terminal from the link kmlucy posted |
I've read the workarounds you guys posted, but it seems to me its for non-hass.io versions of HA. Has anyone found success adopting the workaround in Hass.io? |
I'm in the same boat. How do we work around this for HASSIO? |
I would imagine it's using the same sort of dependencies somewhere in the virtual directory for hassio. Unfortunately I have no experience with hassio atm |
Thanks alagarth. Appreciate the quick response. We will have to see what comes about with SleekXMPP. |
Is this a BUG with the sleekxmpp or is the ecovac addon using an outdated or depreciated method within sleekxmpp? |
For what it's worth, I do a clean download-and-build with this code every day to run my vacuum. I don't get this error ever. So I believe this is something specific to HomeAssistant users. I'm happy to take a patch that fixes the problem, though. |
@wpietri With your setup, what version of sleekxmpp do you use? |
Looking at the logs from the last nightly build, I see this: |
Home Assistant has moved from sleekxmpp to slixmpp==1.4.1. I believe this is where the problems comes from. I, and others, have been able to get it working again by reverting to sleekxmpp temporarily. |
@wpietri: The issue is related to the way timestamps are coming from or being sent to the XMPP server. I find it entirely likely that this could be an issue experienced differently on different combinations of OS and hardware. On my dev machine, sucks works fine via Home Assistant, but on my production Raspberry Pi, it does not. :( If you look at the comments on the issue in SleekXMPP you can some other users outside of Home Assistant that have been experiencing this issue: fritzy/SleekXMPP#478 The second comment has a person experiencing it on only one out of a pool of servers, so there's something weird going on. There are a bunch more people reporting having the issue here in a pull request that was rejected: fritzy/SleekXMPP#461 I don't know enough about XMPP to make an authoritative call here, which is the only reason I haven't pinned sucks to XMPP 1.3.2 (before the issue was introduced.) |
Wow! What a community! I was digging around @fritzy/SleekXMPP some this AM and I saw the same thing. I really wish I could contribute here, but I'm not a developer. I do know scripting languages though and currently learning Python. I do want to say thank you for the time and effort to put this addon together. rschoolm |
@kmlucy Can you link me to where discussion on this suspicion is taking place? I'd like to dig in on why slixmpp might be affecting what SleekXMPP is doing. |
I had problems starting from 0.81.0, which corresponds to when they switched. By manually reverting, I was able to get everything working again. This PR seems to be the change that triggered this for me: home-assistant/core#17283 |
I'm using HassIO and I don't know how to go back to 1.3.2 or 1.3.1... can someone point me on how to do it? |
i did some digging and noticed that the xmpp is a pretty minor part of this library... can someone with better python chops just swap the xmpp library to one that works? |
All of the communication with the vacuum itself is over XMPP, so in some sense it's the heart of it. When I wrote this I looked at a variety of options, and this seemed like the best of a shaky lot. But if somebody would like to take a swing at trying a different library, I'd be very interested to see how it turns out. |
Same problem with XMPP 1.3.3 Edit the file _strptime.py located in ..\Python\Lib\ And add the following code before the line 527 (starting with # If we know the week )
|
@tongueSoftware does this solve this issue, would be great because this also destroyed the ecovacs components for some months. |
for sleekxmpp-1.3.3
|
@slmi I've tried the above code and the date error seems to be fixed. But... Mar 06 13:02:17 hassbian hass[8366]: 2019-03-06 13:02:17 ERROR (read_thread) [sleekxmpp.xmlstream.xmlstream] Connection error. |
https://github.com/fritzy/SleekXMPP/pull/482/files
|
@slmi I'm getting closer :-) Mar 06 21:30:59 hassbian hass[11914]: 2019-03-06 21:30:59 DEBUG (SyncWorker_8) [sucks] got {'code': '0000', 'msg': '操作成功', 'time': 1551904259229, 'data': {'uid': 'XXXXXXXXXX', 'username': 'XXXXX', 'email': 'XXXXXXXXXX@gmail.com', 'country': 'se', 'accessToken': 'XXXXXXXXXX'}} |
Did you figure this out? I have the same issue. |
@Frippuz - how are you pushing these changes to hassio? I'm getting the same error but only on my Raspberry Pi 3 that is running hassio. I would like to try and make a fix myself but I'm not sure how would I push the change to hassio. |
The main code works no? |
AFAIK, there is no work around for hassio yet =/ |
Do you need a 1.3.4 of SleekXMPP? |
It seems that the fixes above helps. But it also seems like I have issues with two libs at the same time. I'm trying to get sucks to work. But it looks like the error wasn't with sucks in the end anyway. My type of 'bot wasn't supported by sucks at all - something that wasn't clear when In started testing it all. There are solutions on the way for that part. I got stuck with the error above. But I think it is more related to sucks and not SleekXMPP itself. To resolve my original error, I downgraded to 1.3.2 and removed the certificate check. I am also running an experimental version of sucks and it works - unsafe, but it works at least. Check this out: https://github.com/bmartin5692/sucks/tree/D901 |
@Frippuz: Thanks for your reply! |
I spent some time last night finally sitting down to truly debug this. I ran into several issues:
I am wondering if it makes sense for us to try switching to Slixmpp, which is a form of SleekXMPP (so hopefully compatible without too many changes) that has had more recent development. It's either that or I will publish a version of SleekXMPP just for sucks that has patches for these issues. |
I have no attachment to any particular XMPP library, and this one has been a thorn in our sides for a long time now, so I'm happy to switch to anything that works better. |
@OverloadUT: Please create PR for your changes... I try to relaunch the dev of the lib, I wait changes from @fritzy and @bear since several months, for a best SleekXMPP... |
@Neustradamus The problem is that I don't have clean changes. I had to just comment out busted stuff and circumvent the cert expiration checks. It would take some time to fix it in such a manner as to be PR-worthy. |
This fork of SleekXMPP disables certificate validation to work around wpietri#62 and wpietri#45 This fix should be seen as a temporary hack until we figure out a long term solution.
The text was updated successfully, but these errors were encountered: