gzip middleware keeps Content-Length: EXR preview fails with ERR_CONTENT_LENGTH_MISMATCH

jerry commented

2026-10-23 13:29:17 +00:00

Contributor

EXR Preview modal always shows "Failed to load preview image." Browser console: net::ERR_CONTENT_LENGTH_MISMATCH on /api/jobs/{id}/files/{fileId}/preview-exr.

Root cause: gzipMiddleware (internal/manager/manager.go ~L471) gzips the body but leaves the handler's Content-Length (uncompressed size). Proof on 0.0.11:

curl without Accept-Encoding: 200, full PNG, fine
curl -H Accept-Encoding:gzip: Content-Encoding gzip + Content-Length 6820608; curl --compressed exits 18 (partial transfer)

Fix: drop Content-Length in the gzip wrapper before WriteHeader, and skip gzip for already-compressed types (image/png, zip, file downloads). Likely affects every endpoint that sets Content-Length.

Done = preview loads in Chrome/Brave, curl --compressed exits 0, release cut. Filed by Jerry for Justin.

EXR Preview modal always shows "Failed to load preview image." Browser console: net::ERR_CONTENT_LENGTH_MISMATCH on /api/jobs/{id}/files/{fileId}/preview-exr. Root cause: gzipMiddleware (internal/manager/manager.go ~L471) gzips the body but leaves the handler's Content-Length (uncompressed size). Proof on 0.0.11: - curl without Accept-Encoding: 200, full PNG, fine - curl -H Accept-Encoding:gzip: Content-Encoding gzip + Content-Length 6820608; curl --compressed exits 18 (partial transfer) Fix: drop Content-Length in the gzip wrapper before WriteHeader, and skip gzip for already-compressed types (image/png, zip, file downloads). Likely affects every endpoint that sets Content-Length. Done = preview loads in Chrome/Brave, curl --compressed exits 0, release cut. Filed by Jerry for Justin.

ash was assigned by eva

2026-10-23 13:29:42 +00:00

eva commented

2026-10-23 13:29:42 +00:00

Owner

Clear spec and repro, so this goes straight to implementation. Keep the fix in the gzip wrapper (drop Content-Length before WriteHeader, skip already-compressed types), add a test that a gzipped response decodes cleanly, leave the Go version alone, and don't merge yourself. A release cut follows the merge.

ash referenced this issue from a pull request that will close it,

2026-10-23 13:30:46 +00:00

manager: drop stale Content-Length when gzipping (closes #25) #26

ash referenced this issue from a commit

2026-10-23 13:30:47 +00:00

manager: drop stale Content-Length when gzipping (closes #25)