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

Emulator: Auth with AADV1, update after consent and ARM delegated access #1113

Merged
merged 2 commits into from Nov 20, 2018

Conversation

carlosscastro
Copy link
Member

@carlosscastro carlosscastro commented Nov 20, 2018

Our app configuration has now been updated in terms of consent and ARM delegated access. This PR updates the resouce to the current ARM resource code.

Result: Single-FA and Multi-FA works again for Microsoft and Non-Microsoft users, see below for screenshots.

Pending: PIN, dongle and smart card not in this PR. The Certificate selection and support for PIN, dongle and smart card is in the ccastro/select-client-certificate branch. Now that this PR is going in, we'll merge master to that branch and close the loop on that work.

Demo 1: Hotmail Single-FA

1 hotmail 1 fa

Demo 2: Hotmail 2 FA with authenticator App on cell phone

2 hotmail 2 fa with device

Demo 3: Microsoft Multi-FA selection

3 msft corporate 2fa selection

Demo 4: Microsoft Phone 2 FA in progress

4 msft 2 fa phone

Demo 5: Microsoft account sign in success

5 success

@coveralls
Copy link

coveralls commented Nov 20, 2018

Pull Request Test Coverage Report for Build 1406

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 55.902%

Totals Coverage Status
Change from base Build 1404: 0.0%
Covered Lines: 4515
Relevant Lines: 7267

💛 - Coveralls

Copy link
Contributor

@justinwilaby justinwilaby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@justinwilaby justinwilaby merged commit 24ffcd5 into master Nov 20, 2018
@justinwilaby justinwilaby deleted the ccastro/authv2 branch November 20, 2018 16:46
carlosscastro added a commit that referenced this pull request Nov 20, 2018
This reverts commit 24ffcd5, reversing
changes made to 3a858de.
carlosscastro added a commit that referenced this pull request Nov 20, 2018
Revert "Merge pull request #1113 from Microsoft/ccastro/authv2"
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

Successfully merging this pull request may close these issues.

None yet

3 participants