-
Notifications
You must be signed in to change notification settings - Fork 107
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
Make ssl certs and keys accessible through RESTAPI #697
Conversation
Attached issue: https://pulp.plan.io/issues/6691 |
CHANGES/6691.removal
Outdated
@@ -0,0 +1 @@ | |||
Removed SecretCharField from the codebase. |
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.
This probably needs to go in CHANGES/plugin_api.
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.
Oh, we never exposed it via the plugin api. Ignore 😺
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 did not see it exposed in the plugin api.
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 it might be better to say here that fields that returned sha256 (ca_cert, client_cert, client_key) now return the full value.
b454dae
to
4435118
Compare
4bbe556
to
945e354
Compare
@mdellweg The code all looks good, my only suggestions are around the changelogs.
|
945e354
to
c34a7ae
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.
This looks right to me. Thank you @mdellweg !
fixes #6691
https://pulp.plan.io/issues/6691