Cannot register new device

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

72.15k stars 30.19k forks source link

Cannot register new device #118931

Open ordor2k opened 4 months ago

ordor2k commented 4 months ago

The problem

after putting my phone number i get "Unexpected error" message.

What version of Home Assistant Core has the issue?

core-2024.6.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

No response

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

joostlek commented 4 months ago

Please give more context, what are you trying to do? With this amount of information we can't help you

ordor2k commented 3 months ago

Just open the addon and try to register a new device. after entering my phone number it gives an error

ordor2k commented 3 months ago

2024-06-09_11-36-25

ordor2k commented 3 months ago

Logger: homeassistant.components.tami4.config_flow Source: components/tami4/config_flow.py:46 integration: tami4 (documentation, issues) First occurred: 11:36:12 AM (1 occurrences) Last logged: 11:36:12 AM

Unexpected exception Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/tami4/config_flow.py", line 46, in async_step_user await self.hass.async_add_executor_job( File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/Tami4EdgeAPI/Tami4EdgeAPI.py", line 215, in request_otp if not response["success"]:


KeyError: 'success'

joostlek commented 3 months ago

Just open the addon

You never mention what integration you are talking about, how are we supposed to help you if you don't provide the actual place where it goes wrong? We have 1260+ integrations, there's no way we can guess which one you're talking about.

From your logs I see tami4. They had a breaking change in their API.

home-assistant[bot] commented 3 months ago

Hey there @guy293, mind taking a look at this issue as it has been labeled with an integration (tami4) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of `tami4` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign tami4` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

_{^{(message by CodeOwnersMention)}}

tami4 documentation tami4 source _{^{(message by IssueLinks)}}

Guy293 commented 3 months ago

Hi, a fix has been merged already, please wait for the new Home Assistant update release.

duvidw commented 3 months ago

I have the same problem in the latest Homeassistant The device is online I can see it via the application on the android smartphone. Homeassistant version: Core 2024.6.2 Supervisor 2024.06.0 Operating System 12.3 Frontend 20240610.0

Guy293 commented 3 months ago

I have the same problem in the latest Homeassistant The device is online I can see it via the application on the android smartphone. Homeassistant version: Core 2024.6.2 Supervisor 2024.06.0 Operating System 12.3 Frontend 20240610.0

Please wait for a release where the fix is included in.

AmichaiHadad commented 3 months ago

@Guy293 Hey im having the exact same issue, waited for the next release of HA as mentioned in this thread, it was released yesterday on june 21st (2024.6.4) and the issue is still the same, was the update not pushed to this version?

Guy293 commented 3 months ago

Please refer to this thread: https://github.com/home-assistant/core/issues/113167#issuecomment-2178501391 @homeassistant close

AmichaiHadad commented 3 months ago

there is still a problem with this integration. now when adding a new tami4 integration it asks for the phone number. when entered it no longer fails on that step and i do get the SMS with the otp code, once i enter it into the setup prompt i get "an unknown error occured". im using HA 2024.7.0

AmichaiHadad commented 3 months ago

same issue persists after updating to 2024.07.01 for the record

Guy293 commented 2 months ago

Hi @AmichaiHadad, I'm currently debugging this issue, you can follow up here: https://github.com/home-assistant/core/issues/121675