woops, move this to process function

itchio · Nov 14, 2024 · e8a16a6 · e8a16a6
1 parent ed434c5
commit e8a16a6
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/zipserver/slurp_handler.go b/zipserver/slurp_handler.go
@@ -25,11 +25,6 @@ func slurpHandler(w http.ResponseWriter, r *http.Request) error {
 		return err
 	}
 
-	if !slurpLockTable.tryLockKey(key) {
-		return fmt.Errorf("Key is currently being processed: %s", key)
-	}
-	defer slurpLockTable.releaseKey(key)
-
 	slurpURL, err := getParam(params, "url")
 	if err != nil {
 		return err
@@ -49,6 +44,11 @@ func slurpHandler(w http.ResponseWriter, r *http.Request) error {
 	}
 
 	process := func(ctx context.Context) error {
+		if !slurpLockTable.tryLockKey(key) {
+			return fmt.Errorf("Key is currently being processed: %s", key)
+		}
+		defer slurpLockTable.releaseKey(key)
+
 		getCtx, cancel := context.WithTimeout(ctx, time.Duration(globalConfig.FileGetTimeout))
 		defer cancel()