home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.71k stars 30.83k forks source link

ORU integration no longer works #36319

Closed dacmcbibs closed 4 years ago

dacmcbibs commented 4 years ago

The problem

Environment

Problem-relevant configuration.yaml

Orange and Rockland meter

## Traceback/Error logs
<!--
  If you come across any trace or error logs, please provide them.
-->

```txt

Additional information

It appears that the site now requires some kind of token or API key... the old URL was : https://oru.opower.com/ei/edge/apis/cws-real-time-ami-v1/cws/oru/meters/' + self.meter_id + '/usage'

After analyzing the data on their website it looks like it is now ;

'https://oru.opower.com/ei/edge/apis/cws-real-time-ami-v1/cws/oru/accounts/'+ some,account.guid '+ /meters/'+'self.meter_id'+'/usage'

I didn't attempt to fix, as i haven't yet determined how to generate a valid API key , but assuming that you need to generate one by authenticating to the site

probot-home-assistant[bot] commented 4 years ago

Hey there @bvlaicu, mind taking a look at this issue as its been labeled with a integration (oru) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

stale[bot] commented 4 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

hirshy commented 3 years ago

I just tried installing this integration and got the following response in my logs: Unexpected oru meter error: ('Error in getting the meter data: %s', {'details': 'HTTP 404 Not Found (2e28eb92-d520-40b4-8e88-bceced194d92)'})

Taking a look at the https://github.com/bvlaicu/oru package you can see that the API request now requires many more credentials (not just the meter number). That's besides for the repo being deprecated and replaced by https://github.com/bvlaicu/coned.

This integration no longer works in its current state and needs to be fixed or removed.

michaeldauria commented 3 years ago

looks like we should be using the addon by the same author instead: https://github.com/bvlaicu/home-assistant-addons/tree/master/oru