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

Allow anonymous notary access #159

Merged
merged 6 commits into from
Aug 19, 2020
Merged

Allow anonymous notary access #159

merged 6 commits into from
Aug 19, 2020

Conversation

jeffb4
Copy link
Contributor

@jeffb4 jeffb4 commented Aug 10, 2020

Allow access to anonymous notary servers (those without authentication turned on). I found this to be necessary when running my own notary server for storing signatures of images in ECR.

I did this by suffixing username "" password "" to the end of the real credentials array, and having methods that hit oauth and compose authentication headers be aware of this.

Opening in draft to get feedback on the approach while I'm continuing to test.

@jeffb4 jeffb4 marked this pull request as draft August 10, 2020 23:03
@jeffb4 jeffb4 marked this pull request as ready for review August 10, 2020 23:42
@jeffb4
Copy link
Contributor Author

jeffb4 commented Aug 10, 2020

This tests successfully with my cluster-local anonymous notary server.

@jeffb4
Copy link
Contributor Author

jeffb4 commented Aug 13, 2020

In a few weeks I'll be in a new position where I won't be using Portieris anymore, so if there's any feedback on this PR that I can address to make it more mergeable, please send such

Copy link
Member

@sjhx sjhx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@sjhx sjhx merged commit ceaa5ab into IBM:master Aug 19, 2020
@jeffb4 jeffb4 deleted the allow-anonymous-notary-retrieval branch August 20, 2020 16:49
tim-gp pushed a commit to tim-gp/portieris that referenced this pull request Oct 1, 2020
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