# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1725375439 -7200
# Node ID 4113e0465b4b03c0ad0eeeb2620c99172645b245
# Parent  2047b0921877fb68438928abffa450c2bb44eda2
ignore empty keys in QIDO-RS requests

diff -r 2047b0921877 -r 4113e0465b4b Plugin/QidoRs.cpp
--- a/Plugin/QidoRs.cpp	Wed Aug 28 16:12:45 2024 +0200
+++ b/Plugin/QidoRs.cpp	Tue Sep 03 16:57:19 2024 +0200
@@ -132,6 +132,16 @@
       for (uint32_t i = 0; i < request->getCount; i++)
       {
         std::string key(request->getKeys[i]);
+        if (key.empty())
+        {
+          /**
+           * This case can happen in Weasis, which uses the following
+           * URL if doing a basic QIDO-RS request (note the "?&"):
+           * http://localhost:8042/dicom-web/studies?&includefield=00080020,00080030,00080050,00080061,00080090,00081030,00100010,00100020,00100021,00100030,00100040,0020000D,00200010&limit=10&offset=0
+           **/
+          continue;
+        }
+
         std::string value(request->getValues[i]);
         args += " [" + key + "=" + value + "]";