Merge pull request #7436 from alrossi/fix/9.2/frnt-nmsp-policy-undefi…

…ned-bug dcache-frontend: check for defined arguments with namespace qos
dCache · Nov 29, 2023 · fc02003 · fc02003
2 parents 83beb88 + 1b10c15
commit fc02003
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/...s/dcache-frontend/src/main/java/org/dcache/restful/resources/namespace/FileResources.java b/...s/dcache-frontend/src/main/java/org/dcache/restful/resources/namespace/FileResources.java
@@ -443,9 +443,12 @@ public Response cmrResources(
                     pnfsHandler.renameEntry(path.toString(), target.toString(), true);
                     break;
                 case "qos":
-                    String targetQos = reqPayload.getString("target");
-                    String qosPolicy = reqPayload.getString("policy");
-                    Integer qosState = (Integer)reqPayload.get("state");
+                    String targetQos =
+                          reqPayload.has("target") ? reqPayload.getString("target") : null;
+                    String qosPolicy =
+                          reqPayload.has("policy") ? reqPayload.getString("policy") : null;
+                    Integer qosState =
+                          reqPayload.has("state") ? (Integer) reqPayload.get("state") : null;
                     Subject subject = RequestUser.getSubject();
                     if (!useQosService) {
                         new QoSTransitionEngine(poolmanager,