Story #8192
[EPIC] As a user, I can rest easy with all sensitive credentials in the database encrypted at rest
50%
Description
Motivation¶
Pulp stores sensitive credentials, e.g. password for Basic Auth, proxy password, or client key for client certificate based authentication in the database in plaintext. The authentication and authorization mechanisms of Pulp keep these safe, but if the database itself was compromised, or dumped, those secrets would be readable to anyone with a copy.
Proposal¶
Encrypt these credentials when they live in the database and decrypt them when they are used. This encryption would use symmetric encryption with a key stored on the filesystem that is generated at install time. A good option would be Fernet symmetric key encryption from the cryptography library.
Related to Story on Removing Secrets from API Response
The scope of the fields that are encrypted should be the same as those that are no longer returned in the API due to sensitivity. See Story 8202 for the list of fields.
Installation Details¶
The private key will need to be generated at install time. We need to determine where to keep these by default securely. They need to be readable by code without a human involved.
Subtasks
Related issues
History
#3
Updated by bmbouter 6 months ago
- Blocked by Task #8202: Make sensitive fields write_only and stop filtering on them added
#7
Updated by daviddavis 3 months ago
- Quarter set to Q2-2021
#8
Updated by daviddavis 3 months ago
- Status changed from NEW to ASSIGNED
- Assignee set to daviddavis
- Sprint set to Sprint 95
#11
Updated by daviddavis 3 months ago
- Blocked by Task #8704: Installer: create a key for pulp to use when encrypting sensitive db fields added
#12
Updated by daviddavis 3 months ago
- Sprint/Milestone deleted (
3.13.0)
Hoping to still get this into 3.13 but it doesn't need to block 3.13. Confirmed with Ansible they don't need it until their next release.
#13
Updated by daviddavis 3 months ago
- Blocked by deleted (Task #8704: Installer: create a key for pulp to use when encrypting sensitive db fields)
#14
Updated by daviddavis 3 months ago
- Subject changed from As a user, I can rest easy with all sensitive credentials in the database encrypted at rest to [EPIC] As a user, I can rest easy with all sensitive credentials in the database encrypted at rest
#15
Updated by daviddavis 3 months ago
- Status changed from POST to NEW
- Assignee deleted (
daviddavis)
#17
Updated by rchan about 2 months ago
- Sprint changed from Sprint 97 to Sprint 98
#18
Updated by mdepaulo@redhat.com about 2 months ago
The filepath will be: /etc/pulp/certs/database_fields.symmetric.key as the default path or fixed path for installer / container / operator.
We agreed (at a pulp_installer meeting I think) that the pulp certs directory would be used, since private keys for certs are kept under there, and it needs similar permissions.
We didn't agree on a filename, but we were thinking of ending with .symmetric.key ( .key is the suffix for private keys for certs.)
#19
Updated by rchan about 1 month ago
- Sprint changed from Sprint 98 to Sprint 99
#20
Updated by daviddavis about 1 month ago
- Sprint/Milestone set to 3.15.0
Please register to edit this issue
.