ensure path is set on read endpoint

HubSpot · May 9, 2017 · 56fcfe2 · 56fcfe2
1 parent 7d7add7
commit 56fcfe2
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/SingularityService/src/main/java/com/hubspot/singularity/resources/SandboxResource.java b/SingularityService/src/main/java/com/hubspot/singularity/resources/SandboxResource.java
@@ -1,6 +1,7 @@
 package com.hubspot.singularity.resources;
 
 import static com.hubspot.singularity.WebExceptions.badRequest;
+import static com.hubspot.singularity.WebExceptions.checkBadRequest;
 import static com.hubspot.singularity.WebExceptions.checkNotFound;
 import static com.hubspot.singularity.WebExceptions.notFound;
 
@@ -137,6 +138,8 @@ public MesosFileChunkObject read(@ApiParam("The task ID of the sandbox to read f
 
     final SingularityTaskHistory history = checkHistory(taskId);
 
+    checkBadRequest(!Strings.isNullOrEmpty(path), "Must specify 'path'");
+
     final String slaveHostname = history.getTask().getOffer().getHostname();
     final String fullPath = new File(history.getDirectory().get(), path).toString();