API key migration must account for keys that pre-date introduction of the odt_ prefix #4652
Description
Current Behavior
#4566 introduced a migration to the new API key format, which now includes a "public ID" segment.
The migration does not consider keys that do not match the length of prefix + key:
This fails to account for API keys that were generated prior to DT v4.9.0, which is where the odt_ prefix was first introduced. Keys generated prior to that version won't match the migration logic's length check, and thus won't be migrated.
This happened in @msymons' test instance, where the first and oldest key has no public ID assigned (it shows as null), and is not marked as legacy (no yellow triangle):
Steps to Reproduce
N/A
Expected Behavior
API keys that pre-date v4.9.0 should be migrated correctly.
Dependency-Track Version
4.13.0-SNAPSHOT
Dependency-Track Distribution
Container Image
Database Server
N/A
Database Server Version
No response
Browser
N/A
Checklist
- I have read and understand the contributing guidelines
- I have checked the existing issues for whether this defect was already reported