runtime.GOMAXPROCS(0) for when threads is set to 0

Author: leafo

README.md

@@ -27,7 +27,7 @@ More config settings can be found in `zipserver/config.go`.
 
 ## Limits
 
-All limits are configured in `zipserver.json` using the names below. For `extract` and `list`, you can override limits per request via HTTP query parameters. CLI overrides are available for `extract` via flags, and `--threads` can override `ExtractionThreads`. A value of `0` means "unbounded" for all limits except `ExtractionThreads` (which is forced to at least 1).
+All limits are configured in `zipserver.json` using the names below. For `extract` and `list`, you can override limits per request via HTTP query parameters. CLI overrides are available for `extract` via flags, and `--threads` can override `ExtractionThreads`. A value of `0` means "unbounded" for all limits except `ExtractionThreads` (which uses `GOMAXPROCS`, and is forced to at least 1).
 
 | Config field | Applies to | Description | Default |
 | --- | --- | --- | --- |

zipserver/archive.go

@@ -14,6 +14,7 @@ import (
 	"net/http"
 	"os"
 	"path"
+	"runtime"
 	"strconv"
 	"strings"
 	"time"
@@ -305,7 +306,10 @@ func (a *ArchiveExtractor) sendZipExtracted(
 	results := make(chan UploadFileResult)
 	threads := limits.ExtractionThreads
 	if threads < 1 {
-		threads = 1
+		threads = runtime.GOMAXPROCS(0)
+		if threads < 1 {
+			threads = 1
+		}
 	}
 
 	done := make(chan struct{}, threads)