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

Unexpected silent discard of DICOM objects with invalid AccessionID #447

Open
jas88 opened this issue Dec 19, 2024 · 0 comments
Open

Unexpected silent discard of DICOM objects with invalid AccessionID #447

jas88 opened this issue Dec 19, 2024 · 0 comments
Assignees

Comments

@jas88
Copy link
Member

jas88 commented Dec 19, 2024

Describe the bug

Discovered in HIC Dermatology feed, where a small fraction of data has an AccessionID over 16 characters long which violates the DICOM spec and is incompatible with our SQL schema

To Reproduce

Should be reproducible with any DICOM with a long AccessionID value. May also apply to other tags such as PatientName, also seen elsewhere with CodeMeaning strings in SR objects.

Expected behavior

Either import truncated/redacted, or at least log the fact something is discarded clearly rather than silently ignoring!

@jas88 jas88 self-assigned this Dec 19, 2024
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

No branches or pull requests

1 participant