core icon indicating copy to clipboard operation
core copied to clipboard

Ring Doorbell integration does not prompt for MFA verification and fails

Open 3DPixel opened this issue 10 months ago • 5 comments

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

3DPixel avatar Mar 27 '24 07:03 3DPixel

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)

home-assistant[bot] avatar Mar 27 '24 07:03 home-assistant[bot]

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?

sdb9696 avatar Apr 05 '24 13:04 sdb9696

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

williamhenriquez avatar Apr 11 '24 19:04 williamhenriquez

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 avatar May 09 '24 15:05 konikvranik

Additional error related to this:

024-05-09 17:53:26.440 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry [email protected] 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

konikvranik avatar May 09 '24 15:05 konikvranik

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 has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.