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

Registry authentication enforcement #59

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

davidv1992
Copy link

This ensures people need to be logged in for all operations on the registry, ensuring that private code is not leaked. I intend to make a better implementation of this once #57 lands.

@gagbo
Copy link
Collaborator

gagbo commented Jan 27, 2023

I'll check this code when I have time, but looking at the title, it looks like I'd rather wait for #50 (and/or the associated RFC) instead of trying to create and enforce our own security model around crates privacy

@davidv1992
Copy link
Author

Apologies, didn't realize #50 did the same as this PR. This one is also build on the same Cargo RFC. Feel free to close this if a version based on the refactor isn't needed.

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.

2 participants