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

STOW-RS: adjust accepted and returned content type from application/json to application/dicom+json #621

Closed
gunterze opened this issue Apr 10, 2017 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@gunterze
Copy link
Member

No description provided.

@gunterze gunterze added the task label Apr 10, 2017
@gunterze gunterze added this to the 5.10.1 milestone Apr 10, 2017
@gunterze gunterze self-assigned this Apr 10, 2017
gunterze added a commit to dcm4che/dcm4che that referenced this issue Apr 10, 2017
@vrindanayak
Copy link
Member

vrindanayak commented Apr 12, 2017

Storage works with application/json as well. Need below code in StowRS.java

        if (getHeaderParamValue(headerParams, "transfer-syntax") != null) {
          Map<String, String> parameters = new HashMap<>();
          parameters.put("transfer-syntax", "1.2.840.10008.1.2.4.50");
          MediaType mediaType = new MediaType("image", "jpeg", parameters);
        }

because MediaTypes.transferSyntaxOf called from StowRS.java checks for below condition :

        String tsuid = bulkdataMediaType.getParameters().get("transfer-syntax");
           if (tsuid != null)
            return tsuid;

gunterze added a commit that referenced this issue Apr 18, 2017
@gunterze gunterze changed the title STOW-RS: adjust accepted content type from application/json to application/dicom+json STOW-RS: adjust accepted and returned content type from application/json to application/dicom+json Apr 18, 2017
@vrindanayak vrindanayak reopened this Apr 26, 2017
@vrindanayak
Copy link
Member

Support not extended on server side for application/json.

vrindanayak pushed a commit to dcm4che/dcm4che that referenced this issue Jun 8, 2017
hczedik pushed a commit to dcm4che/dcm4che that referenced this issue Jun 26, 2017
@gunterze gunterze added enhancement New feature or request and removed task labels Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants