-
Notifications
You must be signed in to change notification settings - Fork 112
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
Problem: ssl fields on remote are hard to use #99
Conversation
c9e004b
to
fc80217
Compare
Codecov Report
@@ Coverage Diff @@
## master #99 +/- ##
=========================================
+ Coverage 67.79% 67.8% +0.01%
=========================================
Files 65 65
Lines 3018 3019 +1
=========================================
+ Hits 2046 2047 +1
Misses 972 972
Continue to review full report at Codecov.
|
Solution: switch all ssl fields to TextField This patch switches the storage of SSL certs, keys, and CAs from filesystem to database. This patch also introduces a new serializer field that returns a SHA256 digest for GET operations. Required PR: pulp/pulpcore-plugin#91 fixes: pulp#4506 https://pulp.plan.io/issues/4506 re: pulp#4676 https://pulp.plan.io/issues/4676
@@ -1,5 +1,6 @@ | |||
from gettext import gettext as _ | |||
import os | |||
import hashlib |
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.
alphebatize this please.
@@ -209,3 +210,14 @@ def to_representation(self, value): | |||
prefix.strip('/'), | |||
base_path.lstrip('/') | |||
)) | |||
|
|||
|
|||
class SecretCharField(serializers.CharField): |
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.
👍
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 is a great improvement! Thank you @dkliban 👍
Solution: switch all ssl fields to TextField
This patch switches the storage of SSL certs, keys, and CAs from filesystem to
database. This patch also introduces a new serializer field that returns a SHA256
digest for GET operations.
Required PR: pulp/pulpcore-plugin#91
Fixes: #4506
https://pulp.plan.io/issues/4506
Re: #4676
https://pulp.plan.io/issues/4676