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
69.86k stars 28.96k forks source link

Ring Doorbell integration does not prompt for MFA verification and fails #114279

Open 3DPixel opened 3 months ago

3DPixel commented 3 months ago

The problem

There is no obvious way of changing the login credentials in Ring integration if it is already setup so I deleted it and tried to re-add it. It asked me for my Ring login details but kept failing.

I tried to login to my ring.com account directly and I was prompted for an MFA code. The Ring integration does not ask for this so I'm guessing why logging into to the Ring account via HA is failing because it's not prompting to add the MFA code at login.

What version of Home Assistant Core has the issue?

core-2024.3.3

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

Ring Doorbell

Link to integration documentation on our website

https://www.home-assistant.io/integrations/ring

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

home-assistant[bot] commented 3 months ago

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

Code owner commands Code owners of `ring` 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 ring` 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)


ring documentation ring source (message by IssueLinks)

sdb9696 commented 3 months ago

Hi, MFA is working for me. Are you sure you're using the right credentials? Could you enable debug logging for the ring integration and the ring_doorbell library and upload some logs?

williamhenriquez commented 2 months ago

I just enabled Ring integration with my Home Assistant. I have the integration entry, but it appears without devices or entities. I have done the process several times but the error persists.

Home Assistant Core 2024.4.2 Supervisor 2024.04.0 Operating System 12.1 Interfaz 20240404.1

konikvranik commented 2 months ago

I have the same issue. It worked before, but in some version this year it stopped to work.

IMHO this is related to this issue:

2024-05-09 17:49:43.029 WARNING (SyncWorker_4) [urllib3.connectionpool] Retrying (Retry(total=5, connect=0, read=0, redirect=None, status=None)) after connection broken by 'NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fff0cdc7a70>: Failed to establish a new connection: [Errno -3] Try again')': /clients_api/session
konikvranik commented 2 months ago

Additional error related to this:

024-05-09 17:53:26.440 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry xxx@xxx.xxx for ring
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 575, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/ring/__init__.py", line 56, in async_setup_entry
    await devices_coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 280, in async_config_entry_first_refresh
    await self._async_refresh(
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 315, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/ring/coordinator.py", line 60, in _async_update_data
    await _call_api(self.hass, getattr(self.ring_api, update_method))
  File "/usr/src/homeassistant/homeassistant/components/ring/coordinator.py", line 26, in _call_api
    return await hass.async_add_executor_job(target, *args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
asyncio.exceptions.CancelledError: Home Assistant is stopping