Add missing query string to the URI trailing slash redirect#523

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

joseluisq merged 1 commit intomasterfromadd-uri-query-trailing-slash-redirect

Feb 3, 2025

Merged

Add missing query string to the URI trailing slash redirect#523

joseluisq merged 1 commit intomasterfromadd-uri-query-trailing-slash-redirect

Feb 3, 2025

Conversation

Copy link

Collaborator

joseluisq commentedFeb 3, 2025•
edited
Loading

Description

This PR adds the missing query string to the URI when performing a trailing slash redirect only if the feature is enabled.

See feature:https://static-web-server.net/features/trailing-slash-redirect/.

Related Issue

Motivation and Context

How Has This Been Tested?

Before

$ curl -i"http://localhost:8787/assets?param=123"# HTTP/1.1 308 Permanent Redirect# location: /assets/# vary: accept-encoding# cache-control: public, max-age=86400# content-length: 0# date: Mon, 03 Feb 2025 21:19:19 GMT

After

$ curl -i"http://localhost:8787/assets?param=123"# HTTP/1.1 308 Permanent Redirect# location: /assets/?param=123# vary: accept-encoding# cache-control: public, max-age=86400# content-length: 0# date: Mon, 03 Feb 2025 21:19:29 GMT