-
Notifications
You must be signed in to change notification settings - Fork 10
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
Cloud API about to change by the looks of it #11
Comments
thanks for the heads up. I'll see if I can get some time to investigate the new API and update |
It seems the protocol has slightly changed and that has broken things. in get_home() it drops out with error 32: COMMENT It seems that there is an attempt to exclude anything that is not a phone app from calling the API. I cannot replicate the above behaviour anymore. It now drops out with this error 32 regardless if I interact with the phone app at the same time. |
Thanks. I've just started to look at the new API and will hopefully have an update soon. |
See this issue in home assistant for some more discussion recent API issues |
In
the url, headers and data submitted in the request are:
the response is And And it fails with: |
I've pushed https://github.com/ttroy50/pyephember/blob/update_api_docs/API.md which has some of the work from my documentation of the new API. unfortunately this is from a couple of weeks ago and I haven't had a chance to expand on it yet. I'm getting very little time to look into this at the moment. |
Hi @ttroy50, I'd like to help out and contribute to this, but I'm having difficulty capturing the traffic. How are you doing it? I used polarproxy, but it's not able to establish a connection to the mqtt server, so I'm not getting a pcap output for this traffic. It's probably a local issue though. I was able to modify mitmproxy-mqtt-script to give me hex output, but I see your output looks like you're opening it in WireShark. I did find that they might be using EnOS (https://github.com/EnvisionIot) as their IoT manager. It uses port 18883 for MQTTS, when the standard is 8883. So once we can figure out the rest of the parameters used by Ember, we might be able to use that. |
Curious if this still works Im using the following auth format for auth "Authorization: Bearer XXXX" -> where XXXX = "data.token" from login "phoneToken": "really_long_phone_token" I've also noticed its HTTP 2.0 - something postman doesn't yet support, is the a hard requirement? Curious how others are getting on. |
@DarkSlice1 you are passing the auth token slightly incorrectly, and the error message is very opaque. It's just
You just need to drop the bearer, since they are using a custom auth mechanism. |
ah perfect, thank you 👍 |
Point index 10, if value = 2, then the boiler is active and burning oil |
The code in pull request #13 is working for me with the current API. |
The code in pull request 13 works for me (I was looking for get_zone_status() which is now is_zone_boiler_on() but it all seems to work. Thanks. |
If anyone else here could test the code from #13 and let us know if it works it would be very helpful towards getting a new release out. Thanks to @rupertleveneucd for doing the change. |
Hi @ttroy50, I've tested it with the latest version from |
I've tested latest version from master and working for me well. How can we go about getting ephember home assistant updated with this method. |
@ttroy50 I think all that is needed is to bump this master version on pypi.org. The EPHcontrol in home assistant already pull this as a requirement in it's manifest file. It's currently pulling version 3.9. So we can clone that to a custom_component and test there. |
Is it not version 0.3.1? The manifest looks for 'pyephember 0.3.1'? Where do you get the 3.9 version number (Maybe I look at the wrong place). Is there a way to place the correct library with a custom_component to bypass the update on pypi.org? I am happy to test. |
You are correct 0.3.1 apologies. I have my own custom component but the Requirements paramater in the manifest.json no longer allow you to specify a GIT. Has to come from pypi.https://www.google.com/url?sa=t&source=web&rct=j&url=https://developers.home-assistant.io/docs/creating_component_code_review/&ved=2ahUKEwjhjPbOwZD8AhVTZ8AKHUxvDbYQFnoECA0QAQ&usg=AOvVaw0yDfe2nFRX8nPONLZZQ3wc |
Not yet familiar with home assistant, currently try to switch from openhab to that. Is it not possible to have the library directly within the custom_component and not use the requirements mechanism to retrieve that from pypi.org. Ideally an update in pypi.org is better but as a solution in the meantime. |
ok I was able to do a pip install in the home assistant container directly. The method from here https://developers.home-assistant.io/docs/creating_integration_manifest/ Then the custom component loaded but threw some errors around a is_hot_water so I removed all entries around that. |
Some of the api has changed. For example get_zone_status() which is now is_zone_boiler_on() and the is_hot_water does not exist anymore; so some adjustment might be needed to get everything working. Not sure how you managed to install this, I need to spend some time to figure out how to do the pip install on a docker container I am running for this ... |
add the standard install to you config file.
Create a folder under custom_componets called ephember. Needs to be the same name to override the standard component. Remove the version from the requirement in manifest.json Get to the homeassistant container console. In my case through portioner as I have HASSOS
Restart HA and check the log file of errors. Would need someone who understands the API to adjust for the changes since the last time this was working was 2 years ago. |
It took me a while. I have a docker container, so no ssh and no shell; needed to learn you can use 'docker exec' to execute commands in the container and then did the above. Run into some issues. I did not understand what you mean exactly by 'Remove the version from the requirement in manifest.json'. So instead I set the version to 0.4.0 which is the new installed pyephember. I restarted home assistant and got these two errors:
So it seems it wants to install the old version again, how do you prevent this? ---> Found out the manifest.json needs a version number otherwise home assistant takes the original version and not the new one in custom_components |
I think your custom component isn't getting picked up. Hence the default
component is complaining.
Did you add a version param to the manifest file ?
…On Fri 23 Dec 2022, 22:42 UtzR, ***@***.***> wrote:
It took me a while. I have a docker container, so no ssh and no shell;
needed to learn you can use 'docker exec' to execute commands in the
container and then did the above. Run into some issues.
I did not understand what you mean exactly by 'Remove the version from the
requirement in manifest.json'. So instead I set the version to 0.4.0 which
is the new installed pyephember.
I restarted home assistant and got these two errors:
Platform error: climate - Requirements for ephember not found:
['pyephember==0.3.1'].
22:31:13 – (ERROR) config.py
Unable to install package pyephember==0.3.1: ERROR: Could not install
packages due to an OSError: [Errno 13] Permission denied: '/.local' Check
the permissions. [notice] A new release of pip available: 22.3 -> 22.3.1
[notice] To update, run: pip install --upgrade pip
22:31:13 – (ERROR) util/package.py - message first occurred at 22:30:33
and shows up 3 times
So it seems it wants to install the old version again, how do you prevent
this?
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALVJ7EO3W7OLDP64ECAKPCLWOYTGVANCNFSM4VOKRRFA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
OK some more time to play with it today so I have fixed up the boost (aux), set_temperature and set_mode functions. Original code: https://github.com/home-assistant/core/blob/dev/homeassistant/components/ephember/climate.py
|
just tested your code and that works. Thanks. I will experiment with it for a bit more ... |
@ttroy50 could you bump the Master 4.0 version to the latest on PyPi ? https://pypi.org/project/pyephember/ |
I have device type 4 on my immersion sensor and 2 on my normal rooms if that's any help.
|
Sorry for not getting back. I’ve started to add some of you as collaborators on the project to allow you to make updates |
Thanks @ttroy50. Can you please also give someone maintainer access to the pypi project so that it can be updated? |
I bumped the Pypi to the 4.0 version now after I got maintainer access last night |
So would just need someone to fix up climate.py on the offical home assistant component and we are good to go. I am currently using my own Custom Component. Don't feel confident enough to suggest all the changes. https://github.com/home-assistant/core/blob/dev/homeassistant/components/ephember/climate.py |
Hi guys, great work on the API - I was almost certain things were working recently however I just started a fresh copy and I'm getting an error thrown at the following:
Any idea what might be going on? def get_home(self, gateway_id=None):
............... |
In case you didn't get/see this from EPH, presumably relevant...
The text was updated successfully, but these errors were encountered: