Skip to content

feat(MdsV3ServiceImpl): adjusting checkLatestDataExist causing lock#80

Open
hrezend wants to merge 1 commit intoline:mainfrom
hrezend:main
Open

feat(MdsV3ServiceImpl): adjusting checkLatestDataExist causing lock#80
hrezend wants to merge 1 commit intoline:mainfrom
hrezend:main

Conversation

@hrezend
Copy link
Copy Markdown

@hrezend hrezend commented Apr 15, 2026

What is this PR for?

Overview or reasons

  • The checkLatestDataExist method was incorrectly using the DB auto-increment PK (getId()) to compare against the FIDO Alliance BLOB sequence number (metadataBLOBPayload.getNo()), causing the MDS metadata sync to be permanently blocked once the PK value exceeded the BLOB sequence number.
  • image

Tasks

  • Fixed the comparison in checkLatestDataExist to use metadataTocEntity.getNo() (FIDO Alliance BLOB sequence number) instead of getId() (DB auto-increment PK).
  • Added a clarifying comment explaining why getNo() must be used over getId().

Result

  • MDS metadata sync now correctly compares BLOB sequence numbers, preventing the sync from being permanently blocked due to a PK/sequence number mismatch.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 15, 2026

CLA assistant check
All committers have signed the CLA.

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