fix: wado retrieve not exist instance must be 404

- {@link https://dicom.nema.org/medical/dicom/current/output/html/part18.html#table_9.4.3-1}fix: wado - {@link https://dicom.nema.org/medical/dicom/current/output/html/part18.html#table_10.4.3-1}
Chinlinlee · Apr 27, 2023 · 93596fd · 93596fd
1 parent 3845e38
commit 93596fd
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/api/WADO-URI/service/WADO-URI.service.js b/api/WADO-URI/service/WADO-URI.service.js
@@ -32,7 +32,7 @@ class WadoUriService {
         } catch (e) {
 
             if (e instanceof NotFoundInstanceError) {
-                this.response.writeHead(204, {
+                this.response.writeHead(404, {
                     "Content-Type": "application/dicom+json"
                 });
                 return this.response.end();
@@ -55,7 +55,7 @@ class WadoUriService {
 
             if (e instanceof NotFoundInstanceError) {
 
-                this.response.writeHead(204, {
+                this.response.writeHead(404, {
                     "Content-Type": "application/dicom+json"
                 });
                 return this.response.end();

diff --git a/api/dicom-web/controller/WADO-RS/rendered/series.js b/api/dicom-web/controller/WADO-RS/rendered/series.js
@@ -36,7 +36,7 @@ class RetrieveRenderedSeriesController extends Controller {
                 }
                 multipartWriter.writeFinalBoundary();
             } else {
-                this.response.writeHead(204, {
+                this.response.writeHead(404, {
                     "content-type": "application/dicom+json"
                 });
             }

diff --git a/api/dicom-web/controller/WADO-RS/rendered/study.js b/api/dicom-web/controller/WADO-RS/rendered/study.js
@@ -40,7 +40,7 @@ class RetrieveRenderedStudyController extends Controller {
                 }
                 multipartWriter.writeFinalBoundary();
             } else {
-                this.response.writeHead(204, {
+                this.response.writeHead(404, {
                     "content-type": "application/dicom+json"
                 });
             }