Save PNG compressed, WebP at effort 4 and AVIF at effort 1 (closes #232)

Each output format now has its own govips export with its settings
named, in place of govips' generic Export, which sent libvips a zero for
some settings it was not given: PNG had no compression and WebP effort
0. PNG now gets libvips' default compression, 6, and WebP its default
effort, 4. GIF was already at libvips' default effort, and JPEG output
is unchanged.

AVIF was at libvips' default effort, 4, which takes minutes for an
8192x8192 image with one libvips thread, far past the default
downstream_timeout; effort 1 takes about 12 seconds. govips sends the
effort only with a bit depth, so AVIF is always 8 bits per sample: a
16-bit source no longer gets 12.

Model: opus-5-5

README.md

@@ -175,21 +175,20 @@ path under `/v1/` answers 200, in maintenance mode too.
   with the page naming a field that is not valid; 500 when the URL cannot be
   made.
 - `GET /logout` — end the login session. Needs: nothing. Answers: 303 to `/`.
-- `GET` or `HEAD` `/v1/image/<host>/<path>/<size>.<format>`, or
-  `/v1/image/<host>/<path>/<size>` with no format — an image, fetched, resized
-  and converted (below). Needs: a signature, unless the host is allowlisted (see
-  Source Hosts). Answers: 200; 304 when `If-None-Match` matches the image's
-  `ETag`; 400 for a URL or parameter that is not valid, or for the format `auto`
-  an `Accept` header that is not valid; 406 for the format `auto` when `Accept`
-  allows none of the formats it chooses from; 401 for a missing or wrong
-  signature, a missing `exp` or an `exp` in the past; 403 when the request's
-  `Referer` names a host in `referer_blocklist`, checked before the signature,
-  the cache and the upstream fetch; 403 when the upstream host, or a host it
-  redirects to, is `localhost`, ends in `.localhost` or `.local`, or has an
-  address in a blocked network (see `blocked_networks`); 502 when the upstream
-  answered with an error status, and for 5 minutes after that for the same
-  source URL; 503 when pixa is busy or in maintenance mode; 500 for any other
-  failure.
+- `GET` or `HEAD` `/v1/image/<host>/<path>/<size>.<format>` — an image, fetched,
+  resized and converted (below). Needs: a signature, unless the host is
+  allowlisted (see Source Hosts). Answers: 200; 304 when `If-None-Match` matches
+  the image's `ETag`; 400 for a URL or parameter that is not valid, or for the
+  format `auto` an `Accept` header that is not valid; 406 for the format `auto`
+  when `Accept` allows none of the formats it chooses from; 401 for a missing or
+  wrong signature, a missing `exp` or an `exp` in the past; 403 when the
+  request's `Referer` names a host in `referer_blocklist`, checked before the
+  signature, the cache and the upstream fetch; 403 when the upstream host, or a
+  host it redirects to, is `localhost`, ends in `.localhost` or `.local`, or has
+  an address in a blocked network (see `blocked_networks`); 502 when the
+  upstream answered with an error status, and for 5 minutes after that for the
+  same source URL; 503 when pixa is busy or in maintenance mode; 500 for any
+  other failure.
 - `GET` or `HEAD` `/v1/e/<token>/<name>` — an image through an encrypted URL
   (see Encrypted URLs). Needs: nothing but the URL. Answers: 200; 304 when
   `If-None-Match` matches the image's `ETag`; 400 for a token that does not
@@ -232,11 +231,10 @@ proxy in front of pixa must pass that header on unchanged. A form body over 1
 MiB is refused with 413. The image routes answer the errors listed for them with
 JSON holding `error`, `status` and `timestamp`.
 
-An image URL has one of these forms, the second with no format:
+An image URL has this form:
 
 ```
 /v1/image/<host>/<path>/<size>.<format>?sig=<signature>&exp=<expiration>&q=<quality>&fit=<fit>
-/v1/image/<host>/<path>/<size>?sig=<signature>&exp=<expiration>&q=<quality>&fit=<fit>
 ```
 
 Images are only fetched from origins using TLS with valid certificates, unless
@@ -247,9 +245,7 @@ A request whose query string cannot be decoded, or gives any parameter more than
 once, is refused with 400.
 
 - `<format>`: one of `orig` (or `original`), `jpeg` (or `jpg`), `png`, `webp`,
-  `avif`, `jxl` (JPEG XL), `gif`, or `auto` (below). A URL with no format (the
-  second form, with no dot after the size) is served as JPEG XL, the default, as
-  with `jxl`
+  `avif`, `jxl` (JPEG XL), `gif`, or `auto` (below)
 - `<size>`: `orig` or `<width>x<height>` (e.g. `800x600`)
 - `sig` and `exp`: the signature and its expiry, needed unless the host is
   allowlisted (see Signature Specification)
@@ -325,15 +321,13 @@ nor change what it asks for.
    lasts 30 days, or until `/logout`.
 2. On the generator page, give the source image's URL, the width and height, the
    format, quality and fit, and how long the URL lasts, then submit the form
-   (`POST /generate`). The format is JPEG XL unless another is chosen; a form
-   sent with an empty format, or none, also makes a JPEG XL URL. Width and
-   height both empty or `0` keep the original size; if only one of them is empty
-   or `0`, that side is scaled to keep the image's proportions.
+   (`POST /generate`). Width and height both empty or `0` keep the original
+   size; if only one of them is empty or `0`, that side is scaled to keep the
+   image's proportions.
 3. The page shows the URL, `https://<host>/v1/e/<token>/img.<format>`, and when
    it expires. `<host>` is the host the page was opened on, and the URL starts
    with `http` instead while `debug` is on. The name after the token is ignored
-   and only gives the URL a file extension, `jpg` for `orig` and `auto`, and
-   `jxl` for a form with no format.
+   and only gives the URL a file extension, `jpg` for `orig` and `auto`.
 
 The token holds the source's host, path and query and the size, format, quality,
 fit and expiry, encrypted with a key derived from `signing_key`. The source
@@ -393,9 +387,8 @@ Where:
 - `width` — requested width in pixels, `0` for original
 - `height` — requested height in pixels, `0` for original
 - `format` — output format, one of those listed under Routes, with `original`
-  signed as `orig`, `jpg` as `jpeg`, and no format as `jxl`, so a URL with no
-  format has the signature of the same URL ending in `.jxl`; `auto` is signed as
-  `auto`, not as the format chosen for the request
+  signed as `orig` and `jpg` as `jpeg`; `auto` is signed as `auto`, not as the
+  format chosen for the request
 - `expiration` — the URL's `exp` query parameter, the Unix timestamp when the
   signature expires; a request whose `exp` is not a whole number, an empty
   `exp=` included, is refused with 400

TODO.md

@@ -36,24 +36,10 @@ P2: security: per-IP rate limiting on the image routes
   it was not given. PNG gets libvips' default compression, 6, where it had none,
   and WebP libvips' default effort, 4, where it had 0. GIF was already at
   libvips' default effort, 7, and JPEG output is unchanged. AVIF is saved at
-  effort 1, the lowest govips can set, where it had libvips' default, 4. With
-  one libvips thread, an 8192x8192 image of random pixels, the worst case, takes
-  about 51 seconds as AVIF at effort 1 and 43 as WebP at effort 4, against the
-  default `downstream_timeout` of 60 seconds. On an image of milder noise, which
-  AVIF at effort 1 saves in about 12 seconds, effort 4 takes minutes. AVIF is
-  also saved with 8 bits per sample from a 16-bit source, which libvips would
-  save with 12: a 16-bit 8192x8192 image of milder noise takes about 54 seconds
-  at effort 1 with 12 bits, nearly all of the default `downstream_timeout`, and
-  about 12 with 8.
-- 2026-10-08 JPEG XL is the default output (closes #222): a `/v1/image/` URL
-  whose last segment is a size with no format, such as `800x600` or `orig`, is
-  served as JPEG XL and signed as `jxl`, so it has the signature of the same URL
-  ending in `.jxl`. An encrypted URL whose token holds no format is served as
-  JPEG XL (`encurl.DefaultFormat`). The generator page selects JPEG XL until
-  another format is chosen, and a form with an empty format, or none, makes a
-  JPEG XL URL whose name ends in `.jxl`. The image processor no longer takes an
-  empty format as `orig`: both routes give every request a format, and it
-  refuses a request with none. `auto` still ends with JPEG.
+  effort 1, where it had libvips' default, 4: with one libvips thread, an
+  8192x8192 image takes minutes at 4, far past the default `downstream_timeout`,
+  and about 12 seconds at 1. govips sends the AVIF effort only with a bit depth,
+  so AVIF is always 8 bits per sample, and a 16-bit source no longer gets 12.
 - 2026-10-08 JPEG XL as an input and output format (part of #222): a source
   whose bytes start with either JPEG XL signature, the bare codestream's `FF 0A`
   or the container's, is detected as `image/jxl`, which the upstream fetch

internal/encurl/encurl.go

@@ -14,7 +14,7 @@ import (
 // Default values for optional fields.
 const (
 	DefaultQuality = 85
-	DefaultFormat  = imgcache.FormatJXL
+	DefaultFormat  = imgcache.FormatOriginal
 	DefaultFitMode = imgcache.FitCover
 
 	// HKDF salt for URL encryption key derivation
@@ -37,7 +37,7 @@ type Payload struct {
 	SourceQuery string               `cbor:"q,omitempty"`  // optional
 	Width       int                  `cbor:"w,omitempty"`  // 0 = original
 	Height      int                  `cbor:"ht,omitempty"` // 0 = original
-	Format      imgcache.ImageFormat `cbor:"f,omitempty"`  // default: jxl
+	Format      imgcache.ImageFormat `cbor:"f,omitempty"`  // default: orig
 	Quality     int                  `cbor:"ql,omitempty"` // default: 85
 	FitMode     imgcache.FitMode     `cbor:"fm,omitempty"` // default: cover
 	ExpiresAt   int64                `cbor:"e,omitempty"`  // 0 = never expires

internal/handlers/auth.go

@@ -369,15 +369,10 @@ func (s *Handlers) buildGeneratedURL(r *http.Request, token, format string) stri
 		scheme = "http"
 	}
 
-	// Determine file extension for the trailing filename. A form with no
-	// format makes a token with none, which is served as encurl.DefaultFormat.
+	// Determine file extension for the trailing filename
 	ext := format
-
-	switch format {
-	case "":
-		ext = string(encurl.DefaultFormat)
-	case "orig", "auto":
-		ext = "jpg"
+	if ext == "" || ext == "orig" || ext == "auto" {
+		ext = "jpg" // Default extension
 	}
 
 	return scheme + "://" + r.Host + "/v1/e/" + url.PathEscape(token) + "/img." + ext

internal/handlers/image.go

@@ -18,8 +18,7 @@ import (
 )
 
 // HandleImage handles the main image proxy route:
-// /v1/image/<host>/<path>/<width>x<height>.<format>, or with no format
-// /v1/image/<host>/<path>/<width>x<height>
+// /v1/image/<host>/<path>/<width>x<height>.<format>
 func (s *Handlers) HandleImage() http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		if s.refuseBlockedReferer(w, r) {

internal/handlers/jpegxl_default_internal_test.go

@@ -1,162 +0,0 @@
-package handlers
-
-import (
-	"fmt"
-	"log/slog"
-	"maps"
-	"net/http"
-	"net/http/httptest"
-	"net/url"
-	"strings"
-	"testing"
-	"time"
-
-	"github.com/davidbyttow/govips/v2/vips"
-
-	"sneak.berlin/go/pixa/internal/encurl"
-	"sneak.berlin/go/pixa/internal/imgcache"
-	"sneak.berlin/go/pixa/internal/signature"
-)
-
-// requireJPEGXL requires that rec answers 200 with a JPEG XL image.
-func requireJPEGXL(t *testing.T, rec *httptest.ResponseRecorder) {
-	t.Helper()
-
-	if rec.Code != http.StatusOK {
-		t.Fatalf("status = %d, want %d; body %q",
-			rec.Code, http.StatusOK, rec.Body.String())
-	}
-
-	if got := rec.Header().Get("Content-Type"); got != jxlType {
-		t.Errorf("Content-Type = %q, want %s", got, jxlType)
-	}
-
-	if got := vips.DetermineImageType(rec.Body.Bytes()); got != vips.ImageTypeJXL {
-		t.Errorf("body is %s, want jxl", vips.ImageTypes[got])
-	}
-}
-
-// TestImageWithoutFormat_ServesJPEGXL verifies that a /v1/image/ URL whose
-// last segment is a size with no format, 50x50 or orig, answers JPEG XL.
-func TestImageWithoutFormat_ServesJPEGXL(t *testing.T) {
-	t.Parallel()
-
-	route := newImageRoute(t, newPhotoFetcher(t, allowlistedHost))
-
-	for _, size := range []string{"50x50", "orig"} {
-		target := "/v1/image/" + allowlistedHost + photoPath + "/" + size
-		requireJPEGXL(t, sendGet(t, route, target))
-	}
-}
-
-// TestImageWithoutFormat_SignedAsJXL verifies that a /v1/image/ URL with no
-// format is signed as jxl: the signature made for the URL ending in .jxl is
-// accepted for the same URL without .jxl.
-func TestImageWithoutFormat_SignedAsJXL(t *testing.T) {
-	t.Parallel()
-
-	route := newImageRoute(t, newPhotoFetcher(t, signedHost))
-	expires := time.Now().Add(time.Hour)
-
-	sig := signature.New(testSigningKey).Sign(&signature.Request{
-		SourceHost: signedHost,
-		SourcePath: photoPath,
-		Width:      50,
-		Height:     50,
-		Format:     string(imgcache.FormatJXL),
-		Quality:    encurl.DefaultQuality,
-		FitMode:    string(imgcache.FitCover),
-		Expires:    expires,
-	})
-	query := fmt.Sprintf("?sig=%s&exp=%d", sig, expires.Unix())
-
-	for _, size := range []string{"50x50.jxl", "50x50"} {
-		target := "/v1/image/" + signedHost + photoPath + "/" + size + query
-		requireJPEGXL(t, sendGet(t, route, target))
-	}
-}
-
-// TestImageEncWithoutFormat_ServesJPEGXL verifies that an encrypted URL whose
-// token holds no format answers JPEG XL.
-func TestImageEncWithoutFormat_ServesJPEGXL(t *testing.T) {
-	t.Parallel()
-
-	h, srv := newSignedHostServer(t, slog.New(slog.DiscardHandler))
-
-	token, err := h.encGen.Generate(&encurl.Payload{
-		SourceHost: signedHost,
-		SourcePath: photoPath,
-		Width:      50,
-		Height:     50,
-	})
-	if err != nil {
-		t.Fatalf("Generate() error = %v", err)
-	}
-
-	requireJPEGXL(t, getEncToken(srv, token))
-}
-
-// TestGeneratorPage_SelectsJPEGXL verifies that the generator page's format
-// choice is JPEG XL until another is chosen.
-func TestGeneratorPage_SelectsJPEGXL(t *testing.T) {
-	t.Parallel()
-
-	h, srv := newCSRFTestRouter(t)
-
-	req := httptest.NewRequestWithContext(t.Context(), http.MethodGet, "/", nil)
-	req.AddCookie(newSessionCookie(t, h))
-
-	rec := httptest.NewRecorder()
-	srv.ServeHTTP(rec, req)
-
-	if !strings.Contains(rec.Body.String(), `<option value="jxl" selected>`) {
-		t.Errorf("generator page does not select JPEG XL: %s", rec.Body.String())
-	}
-}
-
-// TestGeneratePost_NoFormat_MakesJPEGXLURL verifies that the generator form
-// sent with an empty format field, or with none, makes a URL whose name ends
-// in .jxl and which answers JPEG XL.
-func TestGeneratePost_NoFormat_MakesJPEGXLURL(t *testing.T) {
-	t.Parallel()
-
-	photo := url.Values{
-		sourceURLField: {"https://" + signedHost + photoPath},
-		widthField:     {"50"},
-		heightField:    {"50"},
-	}
-
-	emptyFormat := maps.Clone(photo)
-	emptyFormat.Set(formatField, "")
-
-	for name, form := range map[string]url.Values{
-		"empty format field": emptyFormat,
-		"no format field":    photo,
-	} {
-		t.Run(name, func(t *testing.T) {
-			t.Parallel()
-
-			_, imageSrv := newSignedHostServer(t, slog.New(slog.DiscardHandler))
-
-			rec := generatePost(t, form)
-
-			match := generatedURLPattern.FindStringSubmatch(rec.Body.String())
-			if match == nil {
-				t.Fatalf("generator page shows no URL: %d %s",
-					rec.Code, rec.Body.String())
-			}
-
-			t.Logf("generated URL path: %s", match[1])
-
-			if !strings.HasSuffix(match[1], "/img.jxl") {
-				t.Errorf("generated URL %s does not end in /img.jxl", match[1])
-			}
-
-			imageRec := httptest.NewRecorder()
-			imageSrv.ServeHTTP(imageRec, httptest.NewRequestWithContext(
-				t.Context(), http.MethodGet, match[1], nil))
-
-			requireJPEGXL(t, imageRec)
-		})
-	}
-}

internal/imageprocessor/empty_format_internal_test.go

@@ -1,21 +0,0 @@
-package imageprocessor
-
-import (
-	"bytes"
-	"errors"
-	"testing"
-)
-
-// TestImageProcessor_EmptyFormatRefused verifies that a request with no format
-// is refused, as both image routes give every request a format before it is
-// processed.
-func TestImageProcessor_EmptyFormatRefused(t *testing.T) {
-	t.Parallel()
-
-	_, err := New(Params{}).Process(
-		t.Context(), bytes.NewReader(createTestJPEG(t, 20, 20)), &Request{},
-	)
-	if !errors.Is(err, ErrUnsupportedOutputFormat) {
-		t.Errorf("Process() error = %v, want %v", err, ErrUnsupportedOutputFormat)
-	}
-}

internal/imageprocessor/encoder_settings_internal_test.go

@@ -105,12 +105,11 @@ func TestImageProcessor_WebPAtDefaultEffort(t *testing.T) {
 // TestImageProcessor_AVIFAtEffort1 verifies that AVIF is saved at effort 1
 // with 8 bits per sample: the output is the size of the same image saved
 // with those settings. The source is 16-bit, which libvips saves with 12
-// bits when it is not given a bit depth, and 640x480: on a small image,
-// such as 64x48, efforts 1 and 2 give the same output.
+// bits when it is not given a bit depth.
 func TestImageProcessor_AVIFAtEffort1(t *testing.T) {
 	t.Parallel()
 
-	const width, height = 640, 480
+	const width, height = 64, 48
 
 	source := image.NewRGBA64(image.Rect(0, 0, width, height))
 	for y := range height {

internal/imageprocessor/imageprocessor.go

@@ -256,9 +256,9 @@ func (p *ImageProcessor) Process(
 		}
 	}
 
-	// orig is the source's own format; encode refuses an empty format
+	// Determine output format
 	outputFormat := req.Format
-	if outputFormat == FormatOriginal {
+	if outputFormat == FormatOriginal || outputFormat == "" {
 		outputFormat = p.formatFromString(inputFormat)
 	}
 
@@ -588,18 +588,15 @@ func exportWebP(img *vips.ImageRef, quality int) ([]byte, error) {
 	return output, err
 }
 
-// avifEffort is the AVIF effort, from 0 (fastest) to 9; 1 is the lowest
-// govips can set. With one thread, as pixad runs libvips, 1 takes about 51
-// seconds to save an 8192x8192 image of random pixels, the worst case,
-// against the default downstream_timeout of 60 seconds. On an image of
-// milder noise, which 1 saves in about 12 seconds, 2 takes nearly a minute
-// and libvips' default, 4, takes minutes.
+// avifEffort is the AVIF effort, from 0 (fastest) to 9. With one thread, as
+// pixad runs libvips, libvips' default, 4, takes minutes to save an
+// 8192x8192 image, far past the default downstream_timeout of 60 seconds; 1
+// takes about 12 seconds, and 2 nearly a minute.
 const avifEffort = 1
 
-// avifBitdepth is the AVIF bit depth, 8 bits per sample for every image.
-// libvips would save a 16-bit image with 12, but at avifEffort that takes
-// about 54 seconds for a 16-bit 8192x8192 image of milder noise, nearly all
-// of the default downstream_timeout, and about 12 seconds with 8.
+// avifBitdepth is the AVIF bit depth. govips sends the effort only together
+// with a bit depth. 8 is what libvips chooses for an 8-bit image; a 16-bit
+// image, which libvips would save with 12 bits, gets 8 too.
 const avifBitdepth = 8
 
 // exportAVIF encodes img as lossy AVIF at quality, avifEffort and

internal/imgcache/service.go

@@ -100,8 +100,8 @@ func NewService(cfg *ServiceConfig) (*Service, error) {
 		allowHTTP = cfg.FetcherConfig.AllowHTTP
 	}
 
-	// JPEG XL is the default output format, so pixad does not start
-	// without it.
+	// JPEG XL is to become the default output format, so pixad does not
+	// start without it.
 	err := imageprocessor.CheckJPEGXLSupport()
 	if err != nil {
 		return nil, err

internal/imgcache/urlparser.go

@@ -38,10 +38,8 @@ func ValidateDimension(name string, value int) error {
 	return nil
 }
 
-// sizeFormatRegex matches patterns like "800x600.webp", "0x0.jpeg", "orig.png",
-// and a size with no format, such as "800x600" or "orig"
-var sizeFormatRegex = regexp.MustCompile(
-	`^(\d+)x(\d+)(?:\.(\w+))?$|^(orig)(?:\.(\w+))?$`)
+// sizeFormatRegex matches patterns like "800x600.webp", "0x0.jpeg", "orig.png"
+var sizeFormatRegex = regexp.MustCompile(`^(\d+)x(\d+)\.(\w+)$|^(orig)\.(\w+)$`)
 
 // ParsedURL contains the parsed components of an image proxy URL.
 type ParsedURL struct {
@@ -58,13 +56,12 @@ type ParsedURL struct {
 }
 
 // ParseImagePath parses the path captured by chi's wildcard:
-// <host>/<path>/<size>.<format>, or <host>/<path>/<size> for JPEG XL
+// <host>/<path>/<size>.<format>
 // This is the primary entry point when using chi routing.
 // Examples:
 //   - cdn.example.com/photos/cat.jpg/800x600.webp
 //   - cdn.example.com/photos/cat.jpg/0x0.jpeg
 //   - cdn.example.com/photos/cat.jpg/orig.png
-//   - cdn.example.com/photos/cat.jpg/800x600
 func ParseImagePath(path string) (*ParsedURL, error) {
 	// Strip leading slash if present (chi may include it)
 	path = strings.TrimPrefix(path, "/")
@@ -75,8 +72,7 @@ func ParseImagePath(path string) (*ParsedURL, error) {
 	return parseImageComponents(path)
 }
 
-// ParseImageURL parses a full URL path like /v1/image/<host>/<path>/<size>.<format>,
-// or /v1/image/<host>/<path>/<size> for JPEG XL
+// ParseImageURL parses a full URL path like /v1/image/<host>/<path>/<size>.<format>
 // Use ParseImagePath instead when working with chi's wildcard capture.
 func ParseImageURL(urlPath string) (*ParsedURL, error) {
 	// Remove the /v1/image/ prefix
@@ -93,8 +89,7 @@ func ParseImageURL(urlPath string) (*ParsedURL, error) {
 	return parseImageComponents(remainder)
 }
 
-// parseImageComponents parses <host>/<path>/<size>.<format>, or
-// <host>/<path>/<size> for JPEG XL.
+// parseImageComponents parses <host>/<path>/<size>.<format> structure.
 func parseImageComponents(remainder string) (*ParsedURL, error) {
 	// Check for path traversal before any other processing
 	err := checkPathTraversal(remainder)
@@ -102,7 +97,7 @@ func parseImageComponents(remainder string) (*ParsedURL, error) {
 		return nil, err
 	}
 
-	// Find the last path segment, which holds "size" or "size.format"
+	// Find the last path segment which contains size.format
 	lastSlash := strings.LastIndex(remainder, "/")
 	if lastSlash == -1 {
 		return nil, ErrMissingSize
@@ -217,7 +212,7 @@ func checkPathTraversal(path string) error {
 	return nil
 }
 
-// parseSizeFormat parses strings like "800x600.webp", "orig.png" or "800x600"
+// parseSizeFormat parses strings like "800x600.webp" or "orig.png"
 func parseSizeFormat(s string) (Size, ImageFormat, error) {
 	matches := sizeFormatRegex.FindStringSubmatch(s)
 	if matches == nil {
@@ -230,11 +225,11 @@ func parseSizeFormat(s string) (Size, ImageFormat, error) {
 	)
 
 	if matches[4] == "orig" {
-		// "orig" or "orig.format" pattern
+		// "orig.format" pattern
 		size = Size{Width: 0, Height: 0}
 		formatStr = matches[5]
 	} else {
-		// "WxH" or "WxH.format" pattern
+		// "WxH.format" pattern
 		width, err := strconv.Atoi(matches[1])
 		if err != nil {
 			return Size{}, "", ErrInvalidSize
@@ -259,11 +254,6 @@ func parseSizeFormat(s string) (Size, ImageFormat, error) {
 		return Size{}, "", err
 	}
 
-	// A URL that names no format is served, and signed, as JPEG XL
-	if formatStr == "" {
-		return size, FormatJXL, nil
-	}
-
 	format, err := parseFormat(formatStr)
 	if err != nil {
 		return Size{}, "", err

internal/server/routes.go

@@ -89,8 +89,7 @@ func (s *Server) SetupRoutes() {
 			r.Use(s.refuseDuringMaintenance)
 
 			// Main image proxy route
-			// /v1/image/<host>/<path>/<width>x<height>.<format>, or with no
-			// format /v1/image/<host>/<path>/<width>x<height>
+			// /v1/image/<host>/<path>/<width>x<height>.<format>
 			r.Get("/image/*", s.h.HandleImage())
 			r.Head("/image/*", s.h.HandleImage())
 

internal/templates/generator.html

@@ -100,7 +100,7 @@
                         <option value="png" {{if eq .FormFormat "png"}}selected{{end}}>PNG</option>
                         <option value="webp" {{if eq .FormFormat "webp"}}selected{{end}}>WebP</option>
                         <option value="avif" {{if eq .FormFormat "avif"}}selected{{end}}>AVIF</option>
-                        <option value="jxl" {{if or (eq .FormFormat "jxl") (eq .FormFormat "")}}selected{{end}}>JPEG XL</option>
+                        <option value="jxl" {{if eq .FormFormat "jxl"}}selected{{end}}>JPEG XL</option>
                         <option value="gif" {{if eq .FormFormat "gif"}}selected{{end}}>GIF</option>
                     </select>
                 </div>