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

2 Factor Auth for NuGet.org sign in #3252

Closed
harikmenon opened this issue Sep 28, 2016 · 17 comments
Closed

2 Factor Auth for NuGet.org sign in #3252

harikmenon opened this issue Sep 28, 2016 · 17 comments

Comments

@harikmenon
Copy link

harikmenon commented Sep 28, 2016

Status: Reviewed

Spec for this feature available here: https://github.com/NuGet/Home/wiki/2-Factor-Auth-for-NuGet.org-sign-in

Discussions should happen on this issue. Please link other issues with similar asks to this one.

@harikmenon harikmenon changed the title Two Factor Auth on NuGet Gallery 2Factor Auth on NuGet Gallery Sep 28, 2016
@skofman1
Copy link
Contributor

skofman1 commented Dec 9, 2016

@shishirx34 , please cost all the tasks in this epic.

@shishirx34
Copy link
Contributor

Moving to backlog until we get the full story on what needs to be done.

@DixonDs
Copy link
Contributor

DixonDs commented Aug 1, 2017

Are there any plans to implement 2-factor auth in the nearest future on nuget.org?

@skofman1
Copy link
Contributor

skofman1 commented Aug 1, 2017

Hi @DixonDs, 2FA is a high priority item on our backlog, and we are working on a spec for this feature. We will share details with the community once it's ready, and will encourage feedback and ideas.

@karann-msft karann-msft changed the title 2Factor Auth on NuGet Gallery [Feature] 2Factor Auth on NuGet Gallery Aug 9, 2017
@forki
Copy link
Contributor

forki commented Aug 10, 2017

Spec link above is 404

@forki
Copy link
Contributor

forki commented Aug 10, 2017

What does this mean for push and nuget keys?

@skofman1
Copy link
Contributor

@forki , 2FA has no impact on API keys, and will be used only for login to the Gallery. Regarding the 404, checking and will get back to you.

@anangaur
Copy link
Member

@forki I have updated the link to the spec. Remember this is still work in progress and I will post on this issue when decent progress in made on this. Please do continue to post your feedback.

@anangaur anangaur removed the Epic label Sep 12, 2017
@anangaur anangaur changed the title [Feature] 2Factor Auth on NuGet Gallery 2 Factor Auth for NuGet.org sign in Sep 12, 2017
@anangaur
Copy link
Member

Update: The spec is ready for review

@clairernovotny
Copy link
Contributor

Looking at the spec, it doesn't currently show how to link an existing account to AAD. How is that intended to work?

@anangaur
Copy link
Member

@onovotny Clicking on the "Sign in with Microsoft" will lead to a login screen that will redirect to an AAD login if the mail id entered is an AAD account. Nothing else changes. Updated the spec with this detail.

@clairernovotny
Copy link
Contributor

What about the case where the AAD and MSA account are the same email address? Will it "just work" in that case?

As a concrete example, I have my MSA currently attached to my NuGet account but I would want to add/change that to be the AAD account (which uses the same email address).

@anangaur
Copy link
Member

Yepp, this is a grey area. My understanding it that this case should already be handled by the MSA/AAD integration where they ask you to choose either MSA or School/Organization account when you have both. When you try this on a Microsoft service, I think you must be getting these options today?

At NuGet.org if you use these interchangeably, we will link both MSA and AAD accounts to the NuGet.org account. This does have some implication on the policies like whether we show a 2FA policy or say its managed by AAD admin as the account is linked to both?

Note that linking multiple accounts won't be possible in any other scenario (or may be existing ones) except for this specific case.

@maartenba
Copy link
Contributor

Maybe semantics, but the spec does not seem to be about 2FA at all (quote: "At NuGet.org, we do not want to build additional 2-FA", as well as "We will not mandate 2-FA usage for all accounts."), but about federated identity and deprecating user registration.

Other than that sounds good! (one concern: is this all going to be configuration or will other gallery instances be enforced to use this, too?)

@infin8x
Copy link

infin8x commented Sep 13, 2017

@anangaur will there be a separate spec at some point showing how this integrates with the client? Some sort of Credential Provider integration, perhaps - similar to what we do for VSTS (and MyGet does as well)?

@anangaur
Copy link
Member

@maartenba We would like to have enhanced security for all the NuGet.org accounts via 2-FA. There are 2 aspects to it:

  1. 2-FA capability: Either we choose to build the capability or delegate it. We chose the later.
  2. Enforcement: We can either enforce or encourage. We chose to start with not enforcing 2-FA and rather encouraging/recommending it and a way for users/groups to enforce it through Organization policies.

Other than that sounds good! (one concern: is this all going to be configuration or will other gallery instances be enforced to use this, too?)

This should be configuration based so that other gallery instances are not forced to use it. I have updated the spec with this detail.

@anangaur
Copy link
Member

@anangaur will there be a separate spec at some point showing how this integrates with the client? Some sort of Credential Provider integration, perhaps - similar to what we do for VSTS (and MyGet does as well)?

Right. This is more like a NuGet.org only feature. Credential provider and related client authentication mechanism would be different feature in future.

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

No branches or pull requests

10 participants