Bug 54

Summary:	Decide what to do wrt. quoting of multipart answers
Product:	Orthanc	Reporter:	Sébastien Jodogne <s.jodogne>
Component:	Orthanc Core	Assignee:	Sébastien Jodogne <s.jodogne>
Status:	RESOLVED FIXED
Severity:	enhancement
Priority:	---
Version:	unspecified
Hardware:	All
OS:	All

Description Sébastien Jodogne 2020-06-29 15:12:42 CEST

[BitBucket user: Sébastien Jodogne]
[BitBucket date: 2017-07-05.11:38:50]

Important for DICOMweb. Check out:
* https://hg.orthanc-server.com/orthanc/changeset/69b0f4e
* https://hg.orthanc-server.com/orthanc/changeset/b8e07269da72ea31b0f069c95fd9ee2bbf5ad3bb

Comment 1 Sébastien Jodogne 2020-06-29 15:20:15 CEST

[BitBucket user: Alain Mazy]
[BitBucket date: 2017-07-07.15:01:00]

tests performed with: 

- the mainline version of DicomWeb plugin
- Orthanc @69b0f4e & Orthanc @521adcc (the commit before)

tested QIDO-RS, WADO-RS & STOW-RS from Osirix 8.5

Everything works fine with both Orthanc version -> this change has no impact on Osirix Integration.

Comment 2 Sébastien Jodogne 2020-06-29 15:20:16 CEST

[BitBucket user: Alain Mazy]
[BitBucket date: 2017-07-07.15:11:33]

If we do integrate the change, Orthanc will

- work with Osirix
- work with recent Orthanc DicomWeb clients (DicomWeb v0.4 or higher)
- will not work with old Orthanc DicomWeb clients (DicomWeb v0.3 or lower)

I would therefore recommend to have a transition phase where we upgrade the DicomWeb clients now (-> bundle a v0.4 of the DicomWeb plugin with the Orthanc 1.2.1 release)

-> reintegrate the 69b0f4e commit in Orthanc 1.3 (possible breaking change -> for next major update)

Comment 3 Sébastien Jodogne 2020-06-29 15:22:10 CEST

[BitBucket user: Sébastien Jodogne]
[BitBucket date: 2018-09-04.08:10:29]

Fix issue #54 (quoting multipart answers)

→ https://hg.orthanc-server.com/orthanc/changeset/e02af4ca8003