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

[WIP] rfc: allow token exclusion for write 2fa #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

travi
Copy link

@travi travi commented Aug 1, 2018

i still have work to do to shape this proposal out completely, but thought there is enough here to open up for early feedback.

for reference, this is a continuation of the conversation started here

@iarna
Copy link
Contributor

iarna commented Aug 22, 2018

Thank you for writing up such a detailed discussion of your needs.

This is a use-case that we've discussed regarding 2fa internally at npm, and its one that we intend to support. We've not sat down and drawn up exactly what our plans are yet, but something roughly like what's described here is one of the ideas raised previously.

I'm going to leave this open until we do that work and finalize our plans, at which point we'll update this with those plans.

@travi
Copy link
Author

travi commented Aug 23, 2018

great to hear. i apologize for not getting back to this before now to flesh it out a bit further, but it sounds like i should hold off for now.

my team is pretty aggressive with automating things and following continuous deployment of our packages, so i'd be more than happy to help out the conversation any way i can, especially if our flow helps with walking through concrete examples.

looking forward to see where this ends up. thank you for all your efforts and for the interest in input from the community.

@jrjohnson
Copy link

It might be going one step too far for this RFC, but I think limiting the non-MFA token to do one specific thing (publish just one package) would make this more secure.

In my case I would like to automate the publication of a package that is used internally, but would not want to put more popular packages where I have publication rights at risk if the token was exposed. Allowing the creation of a special non-MFA single use publication token would solve this issue, and I think help with several of the contingencies raised by @travi's write up.

Also - It's a promise with no weight, but I would easily be able to justify my organization paying for the private package upgrade if it included this feature. 😄

@darcyclarke darcyclarke added Enhancement new feature or improvement Help Wanted Needs Discussion is pending a discussion Security security related labels Oct 30, 2019
@settings settings bot removed the Help Wanted label Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement new feature or improvement Needs Discussion is pending a discussion Security security related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants