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: Option to authorize registrations from a file in the package repo #330

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

nkottary
Copy link
Member

This lays down the ground work for the plan that is still under discussion in #329. This does not change the existing authorization behaviour of the WebUI so existing installations will not be affected. The file based authorization is enabled by a configuration parameter which is optional.

I've written down the details in the dev docs. We need to update the README.md after this goes live on juliahub.com since users refer to that for help on using Registrator.

@StefanKarpinski
Copy link
Contributor

One comment, let's not call this authorised_registrars.txt, which imo is a bit of an odd name. If this file is going to exist in a lot of repos it seems good to match existing conventions for similar files — how about MAINTAINERS to match the CODEOWNERS file name convention? We should also figure out the file format.

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