Skip to content
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

TP2000-1154 Transaction reordering bug #1352

Merged
merged 8 commits into from
Dec 12, 2024

Conversation

mattjamc
Copy link
Collaborator

@mattjamc mattjamc commented Dec 9, 2024

TP2000-1154 Transaction reordering bug

Why

There is currently a bug where if you reorder transactions and then delete one, it becomes impossible to add more objects to a workbasket. This is due to TAP basing a composite_key off the order of a transaction which can change when transactions are reordered. If reordered and one is deleted TAP tries to create a transaction with a composite_key that already exists.

What

This PR changes how a composite_key is generated. The composite key previously included workbasket pk, order and partition.

It appears that composite_key serves as an application level unique ID for a transaction. It does not seem to be used anywhere so changing its structure should solve this bug and not cause any issues.

Composite key is now set to the workbasket id and a 12 digit hexadecimal string. A UUID cannot be used as this field is limited to 16 characters. Nonetheless, a 48 bit hash provides more than enough security with millions of transactions needed before the likelihood of a duplicate (see hash table on this page).

taric_parsers/importer.py Outdated Show resolved Hide resolved
common/models/transactions.py Outdated Show resolved Hide resolved
workbaskets/models.py Outdated Show resolved Hide resolved
workbaskets/models.py Outdated Show resolved Hide resolved
@mattjamc mattjamc marked this pull request as ready for review December 10, 2024 14:38
@mattjamc mattjamc requested a review from a team as a code owner December 10, 2024 14:38
@mattjamc mattjamc merged commit 22f1e8c into master Dec 12, 2024
8 checks passed
@mattjamc mattjamc deleted the TP2000-1154-Transaction-reordering-bug branch December 12, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants