Skip to content
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

Login error - prompt_mfa #230

Open
Imperial-Guard opened this issue Jan 10, 2025 · 5 comments
Open

Login error - prompt_mfa #230

Imperial-Guard opened this issue Jan 10, 2025 · 5 comments

Comments

@Imperial-Guard
Copy link

Not sure if this is related to the outage what Garmin had last Wednesday, however the addon is unable to connect to Garmin and receiving underneath error:

Traceback (most recent call last): File "/config/custom_components/garmin_connect/__init__.py", line 82, in async_login await self.hass.async_add_executor_job(self._api.login) File "/usr/local/lib/python3.13/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) File "/usr/local/lib/python3.13/site-packages/garminconnect/__init__.py", line 235, in login self.garth.login( ~~~~~~~~~~~~~~~~^ self.username, self.password, prompt_mfa=self.prompt_mfa ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ) ^ TypeError: Client.login() got an unexpected keyword argument 'prompt_mfa'

@fabi3n56
Copy link

same issue here.
it could be cool to update addon with possibility to type mfa code received by mail.
Garmin DON'T give the way to desactivate 2FA access :/

@Broekman
Copy link

Broekman commented Jan 17, 2025

Same for me, added 2FA for ECG access which now required also to access the Garmin integration.

Author has stated in README that 2FA is on the roadmap but unclear when to be developed (if even possible). It is not possible to disable 2FA once enabled via the security center of Garmin, so out of luck for now.

@Broekman
Copy link

Turning off MFA is not possible after enabling ECG, customer support also will not allow it.

The Peloton to Garmin app implemented MFA in 2023, could be used as a reference however it seemed like a lot of work.

philosowaffle/peloton-to-garmin#408
philosowaffle/peloton-to-garmin#410

philosowaffle/peloton-to-garmin#432 (initial implementation)
philosowaffle/peloton-to-garmin#549 (bugfixes)
philosowaffle/peloton-to-garmin#622 (background sync flow support)

@Broekman
Copy link

Perhaps good to close this and continue the conversation on #117

@cyberjunky
Copy link
Owner

@Broekman thanks I will have a look, I could not get the garth library working for MFA with my HA integration it works/worked with the python-garminconnect example.py script version though...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants