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

Add read-only and read-write user auth to Nginx recipe #14321

Closed
wants to merge 2 commits into from

Conversation

MmAaXx500
Copy link

Proposed changes

It is mentioned at the top of the page of the nginx recipe that there somewhere would be an example about read-only and read-write user authentication. This is missing, so I do like to fill this gap with this.
I have also changed the image used by the password file creation example because registry:2 does not have the htpasswd utility.

Related issues

Related: #11060, distribution/distribution-library-image#106 (and probably fixes these)

@netlify
Copy link

netlify bot commented Feb 28, 2022

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: e0c2a1a

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/621cac354725390008e31cda

😎 Browse the preview: https://deploy-preview-14321--docsdocker.netlify.app

@usha-mandya
Copy link
Member

@MmAaXx500 Thanks for the PR. The source files for Registry docs are now in the upstream distribution repository. Could you suggest your changes in https://github.com/distribution/distribution? Thanks. Closing this PR here. Feel free to continue the conversation after I close.

@usha-mandya usha-mandya closed this Mar 4, 2022
@MmAaXx500
Copy link
Author

@usha-mandya Thanks for the info, I'll open my PRs there. If these pages are moved there, I suggest that the "Edit this page" on the right side of the documentation page be changed to point to the new repository.

@usha-mandya
Copy link
Member

Thanks @MmAaXx500. We are looking to revamp the Registry docs section to point users to the upstream repo. :)

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

2 participants