-
-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generate API key #336
Generate API key #336
Conversation
I'm still getting the precondition failed |
Hmm. I only had my units installed 3 days ago (post-API change - disappointing) so I'm using a fresh install/login if that makes a difference - they're working correctly, I have no errors. One part I'm not entirely clear on is the |
The android version has the cfc as well, could be some issue with something not getting encoded correctly maybe special chars in the token |
I did a replay of using the method: |
My Home Assistant container is running in UTC, so it's possible I'm tripping over timezones. Does that change anything for you? |
Yes, that's the issue converting the time to UTC before turning it into a timestamp fixes the issue |
I moved the utc conversion, since the timestamp sent by the client is the local time |
just to make it look like their requests |
@hrrrsn great job! |
It works, thank you! I'm not sure who I should buy the coffee for. :) |
Absolutely fantastic work - thank you very much I would like to buy more than one coffee - but I don't have a credit card Do you have a Paypal account? I think this work should be supported and honored! |
Would it not now be prudent to reduce the polling frequency or even just make that a toggle? |
@codyc1515 there are already options to set the polling rate separate for data and energy and a button to request new data. |
@hrrrsn Great job! Would you mind a short explanation of how you found this out? Was it by reverse engineering, and if so with what tools? I'm currently at work after a good nights sleep but I remember seeing the "Bearer: ", "cfc" and some string manipulation starting at position 9 yesterday, but it would have taken me quite a few more hours to finally put it all together like this. |
@hrrrsn @sockless-coding You are genius and my hero, I have a 4 head Panasonic, I depend on this integration a lot. |
I would also really enjoy seeing a write up or any pointers on how you did this. I’d like to do the same for BYD. |
@codyc1515 depending on how much of the code is located in a native lib, it might not be worth the effort. But you can always throw the lib at https://dogbolt.org/ and see what comes out. |
Fixes #331 , #333, #334