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

Setting FallbackEncoding on AdvancedDicomClientConnectionRequest has no effect on C-Find requests #1642

Open
ayedo opened this issue Aug 17, 2023 · 1 comment · May be fixed by #1780
Open
Labels

Comments

@ayedo
Copy link

ayedo commented Aug 17, 2023

Hello everybody.

First of all, thank you very much for your work on fo-dicom! I think we might have found a bug regarding the application of the fallback encodings.

Describe the bug
We are retrieving modality worklist items from a PACS that does not send a Specific Character Set Attribute. We would like to allow the user to configure what the expected character set might be. We tried setting FallbackEncoding, but it has no effect.

To Reproduce

Send a modality work list c-find request with the FallbackEncoding set to something that is not Encoding.Default to a PACS or test server that does not return the Specific Character Set Attribute, and look at the result, it will always be decoded with Encoding.Default.

If you want you can use the public IHE order manager as a test system that does not return the attribute. I'm not sure if you can access it? I think it uses dcmtk underneath.

I looked at the code, and I think the issue here might be related to this one, but I lack the deep understanding of the library to be sure, and even to create a test case.

Expected behavior
I expect the fallback encoding to be applied.

Environment
Fellow Oak DICOM version: 5.1.1
OS: macOS 12.6.7
Platform: net6.0

@ayedo ayedo added the new label Aug 17, 2023
@gofal
Copy link
Contributor

gofal commented Aug 18, 2023

great someone found a soluntion, but saddly immediatelly closed the issue without doing a PR, so the bug stayed in the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants