Fix rendering

go.mod

@@ -62,7 +62,6 @@ require (
 // Version v1.71.1 is breaking the build.
 replace gopkg.in/DataDog/dd-trace-go.v1 v1.71.1 => gopkg.in/DataDog/dd-trace-go.v1 v1.67.1
 
-// This version had to be pinned because the last few commits did not include any Go changes and it's likely
-// the reason why 'go get -u ./...' is not bumping the version. But they're important because they include
-// new C libraries used by lazypdf.
-replace github.com/nitro/lazypdf/v2 v2.0.0-20250205120532-48aae54bfd96 => github.com/nitro/lazypdf/v2 v2.0.0-20250205171610-bfd709fec6cd
+// lazypdf version needs to be pinned as well because of the dd-trace-go version pinning which prevents the lazypdf version from being
+// updated.
+replace github.com/nitro/lazypdf/v2 v2.0.0-20250205120532-48aae54bfd96 => github.com/nitro/lazypdf/v2 v2.0.0-20250206115348-1829ea1fd728

go.sum

@@ -85,8 +85,8 @@ github.com/mattn/go-isatty v0.0.20 h1:xfD0iDuEKnDkl03q4limB+vH+GxLEtL/jb4xVJSWWE
 github.com/mattn/go-isatty v0.0.20/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
 github.com/mitchellh/mapstructure v1.5.1-0.20231216201459-8508981c8b6c h1:cqn374mizHuIWj+OSJCajGr/phAmuMug9qIX3l9CflE=
 github.com/mitchellh/mapstructure v1.5.1-0.20231216201459-8508981c8b6c/go.mod h1:bFUtVrKA4DC2yAKiSyO/QUcy7e+RRV2QTWOzhPopBRo=
-github.com/nitro/lazypdf/v2 v2.0.0-20250205171610-bfd709fec6cd h1:iNpUVFqOQHdmySemiibLm6r1qY5qYSIExP2+bm7M280=
-github.com/nitro/lazypdf/v2 v2.0.0-20250205171610-bfd709fec6cd/go.mod h1:cAh1CSLNZXNh+PizhypjyoEyWz2cSjcGKe9EUBhEfxA=
+github.com/nitro/lazypdf/v2 v2.0.0-20250206115348-1829ea1fd728 h1:lYbrSO6dULPPxdTBI/mwM+do3M7g1BtcNNtOYGleWA8=
+github.com/nitro/lazypdf/v2 v2.0.0-20250206115348-1829ea1fd728/go.mod h1:cAh1CSLNZXNh+PizhypjyoEyWz2cSjcGKe9EUBhEfxA=
 github.com/opentracing/opentracing-go v1.2.0 h1:uEJPy/1a5RIPAJ0Ov+OIO8OxWu77jEv+1B0VhjKrZUs=
 github.com/opentracing/opentracing-go v1.2.0/go.mod h1:GxEUsuufX4nBwe+T+Wl9TAgYrxe9dPLANfrWvHYVTgc=
 github.com/outcaste-io/ristretto v0.2.3 h1:AK4zt/fJ76kjlYObOeNwh4T3asEuaCmp26pOvUOL9w0=

internal/service/worker.go

@@ -99,7 +99,7 @@ func (w *Worker) Process(
 	}
 
 	storage := bytes.NewBuffer([]byte{})
-	dpi := 96
+	dpi := 72
 	//nolint:gosec,G115
 	err = lazypdf.SaveToPNG(ctx, uint16(page), uint16(width), scale, dpi, bytes.NewBuffer(payload), storage)
 	if err != nil {

internal/transport/handler.go

@@ -153,13 +153,14 @@ func (h handler) metadata(w http.ResponseWriter, r *http.Request) {
 
 // Remove all the parameters, but the token and page, from the path. Other parameters can then be passed to the service
 // without making the url signature invalid.
-func (h handler) urlToVerify(r *http.Request) string {
+func (handler) urlToVerify(r *http.Request) string {
 	q := r.URL.Query()
 	for key := range q {
 		if key == "page" || key == "token" {
 			continue
 		}
 		q.Del(key)
 	}
+	r.URL.RawQuery = q.Encode()
 	return r.URL.String()
 }

internal/transport/handler_test.go

@@ -0,0 +1,46 @@
+package transport
+
+import (
+	"fmt"
+	"net/http"
+	"testing"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestHandlerURLToVerify(t *testing.T) {
+	tests := []struct {
+		path     string
+		expected string
+	}{
+		{
+			path:     "/path",
+			expected: "/path",
+		},
+		{
+			path:     "/path?page=1",
+			expected: "/path?page=1",
+		},
+		{
+			path:     "/path?token=2",
+			expected: "/path?token=2",
+		},
+		{
+			path:     "/path?page=1&token=2",
+			expected: "/path?page=1&token=2",
+		},
+		{
+			path:     "/path?page=1&token=2&scale=3&width=4",
+			expected: "/path?page=1&token=2",
+		},
+	}
+	for i, tt := range tests {
+		t.Run(fmt.Sprintf("Scenario %d", i), func(t *testing.T) {
+			t.Parallel()
+			var h handler
+			req, err := http.NewRequest(http.MethodGet, tt.path, nil)
+			require.NoError(t, err)
+			require.Equal(t, tt.expected, h.urlToVerify(req))
+		})
+	}
+}