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

Is SAML2AWS supported Azure MFA when integrating with ADFS 4.0 (2016) #330

Open
JimmyLS opened this issue Aug 7, 2019 · 5 comments
Open

Comments

@JimmyLS
Copy link

JimmyLS commented Aug 7, 2019

In this article I found for ADFS only VIP and RSA are supported as MFA Provider. Now we are trying to enable Azure MFA as Provider and use it to enforce secondary authentication, but struggled to get it working. We have attempted both Mobile app notification or Verification Code as MFA methods.

May I know if Azure MFA is supported? Or any special steps we missed?

Thank you.

@bsx
Copy link
Contributor

bsx commented Aug 8, 2019

We ran into the same issue with Azure MFA in ADFS. I have a branch (https://github.com/bsx/saml2aws/tree/azure-mfa) where i'm currently working on integrating both app notification and verification code. If you are interessted you can test it out. Once i've cleaned up the branch i'll PR it for integration.

@JimmyLS
Copy link
Author

JimmyLS commented Aug 10, 2019

Oh Thank you very much. We will try it at our environment.
Does it works fine in your environment?

@bsx
Copy link
Contributor

bsx commented Aug 12, 2019

It works in our environment, I have tested it with both the verification code as well as the app notification/approval. I however reverse engineered it mostly from the HTML forms and i'm not sure if they are similar in every implementation. If you could try it out and let me know if it works for you that would be very helpful.
One word of warning: it will currently wait indefinitely if the MFA is set to app approval and the approval does not happen. I plan to add a timeout there as well. Selection of MFA method if multiple methods are available is also not implemented, it will just use whatever method is presented by default.

@FernandoMiguel
Copy link

@bsx i assume this isn't merged yet?

can you merger master into your branch pls so we can build the latest?

@wolfeidau
Copy link
Contributor

@bsx would be good to get a PR for this so others can try it out.

Current setup would still be handy for users.

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