Project

Profile

Help

Task #1751

Unify our checksum type field in our models

Added by jcline@redhat.com over 4 years ago. Updated over 1 year ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

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.

History

#1 Updated by bmbouter over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

#2 Updated by bmbouter over 1 year 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.

#3 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF