Task #1751
closed
Unify our checksum type field in our models
0%
Description
I noticed we have several varieties of checksum type fields in our various models. There is a `checksum_algorithm` field in the platform's LazyCatalogEntry model that uses the following regex to validate the field: `ALG_REGEX = r'(md5|sha1|sha224|sha256|sha384|sha512)'`. However, in the pulp_rpm plugin, all the RPM-related models use a special `ChecksumTypeStringField` which uses a function to sanitize the input. It seems to match a subset of the LazyCatalogEntry field.
We should have a single field type (in platform) that every model uses.
Updated by bmbouter about 5 years ago
- Status changed from NEW to CLOSED - WONTFIX
Updated by bmbouter about 5 years ago
Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.
.