Response `Content-Type` header should include `transfer-syntax`
See original GitHub issueCurrently, the transfer syntax is missing from the response Content-Type header. If multiple Accept headers are supplied, without transfer syntax in the response Content-Type, the caller wouldn’t know which transfer syntax was used. More detail can be found here: http://dicom.nema.org/medical/dicom/current/output/html/part18.html#sect_8.7.9.
We’ll need to add the transfer syntax to the RetrieveResourceResponse being returned in the core layer.
To Reproduce Steps to reproduce the behavior:
- Store an image following these instructions
- Retrieve the image
- Inspect the
Content-Typeheader of the response
Expected behavior
The header should include a portion that reads transfer-syntax={value} where {value} is either a valid Transfer Syntax UID or the token *. This value should match the Accept header in the request.
Actual behavior This portion is not included.
Issue Analytics
- State:
- Created 2 years ago
- Comments:8 (8 by maintainers)

Top Related StackOverflow Question
Closing as the PR has merged - thanks for working through the PR build issues with us @wbalbo!
Thanks, @wbalbo! I’ll take a look tomorrow.