fix: Parse private sequences if they have an undefined length.

cornerstonejs / dicomParser

JavaScript parser for DICOM Part 10 data

MIT License

718 stars 230 forks source link

fix: Parse private sequences if they have an undefined length. #211

Closed yagni closed 2 years ago

yagni commented 2 years ago

Fixes a bug where private sequences with undefined length would attempt to be parsed as items. Also, add a test to verify that private sequences with explicit lengths are skipped.

@chafey @malaterre @Zaid-Safadi Any thoughts on this? I'm throwing away the contents of the sequence to keep parity with how we skip over defined length private sequences. Later we can add a config option to allow parsing private sequences and keep them both in sync.

ohif-bot commented 2 years ago

:tada: This PR is included in version 1.8.18 :tada:

The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: