-
Notifications
You must be signed in to change notification settings - Fork 59
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
Cert auth #851
Cert auth #851
Conversation
8368ec0
to
4cbda54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a good change set. The new code all makes sense to me. Going through it together helped which is why I have no questions, but it all makes sense. It follows what's idiomatic for clowder. The refactors here are good too. I don't really have anything to add except for the minor suggestions in the cache code and errors. LGTM!!!!
4cbda54
to
bdeff37
Compare
/retest |
1 similar comment
/retest |
6141a6a
to
79b206a
Compare
/retest |
1 similar comment
/retest |
1cfe153
to
a2e6453
Compare
a2e6453
to
761bb19
Compare
/retest |
Still to do