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

Custom MFA handler doesn't seem to work? #86

Open
cyberjunky opened this issue Dec 15, 2024 · 3 comments
Open

Custom MFA handler doesn't seem to work? #86

cyberjunky opened this issue Dec 15, 2024 · 3 comments

Comments

@cyberjunky
Copy link
Contributor

cyberjunky commented Dec 15, 2024

Opening a new ticket since the old one was closed and full of api changes.

It seems the garth.login method always returns None when a MFA enabled account is used and return_on_mfa=True

  result = self.garth.login(
      self.username, self.password, return_on_mfa=True
  )

I believe garth.login -> client.login which is http.login which points to sso.login
But http.login doesn't return the same args

Can you shed a light, see what I do wrong, or if there is a bug?

variable title is holding "MFA" in sso.login, but the returned dict never ends up in garth.login (= None)

Running with garth 0.5.2 btw, trying to adapt my example.py

@matin
Copy link
Owner

matin commented Dec 16, 2024

I'll take a look

@cyberjunky
Copy link
Contributor Author

Any luck in finding if there is an issue, and/or fix?

@cyberjunky
Copy link
Contributor Author

@matin I got it working, see PR #92 hope you can merge it, if it can be optimized please do ofcourse!

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

2 participants