Page redirect: Preserve query data in "PAGE" url

[ivankravets]

Details

We use URL in PAGE format in different places. This allows us to control default LOCALIZATION and default VERSION (latest or specific). Also, we add some marketing data via QUERY PARAMETERS to URL for GA.

Example: http://docs.platformio.org/page/projectconf/section_env_build.html?utm_source=platformio&utm_medium=piohome#build-flags

Expected Result

Valid redirect should be: http://docs.platformio.org/en/latest/projectconf/section_env_build.html?utm_source=platformio&utm_medium=piohome#build-flags

Actual Result

http://docs.platformio.org/en/latest/projectconf/section_env_build.html#build-flags

[stsewd]

Can you please share how you configured the redirection? I'll try to replicate this tomorrow.

[ivankravets]

I have only these items:

[stsewd]

I don't see the redirect from the original issue. But I was able to reproduce a different bug.

Are you sure you don't have an extra setup for redirecting from http://docs.platformio.org/page/projectconf/section_env_build.html?utm_source=platformio&utm_medium=piohome#build-flags ?

The another bug:

https://docs.platformio.org/en/latest/userguide/cmd_serialports.html redirects to https://docs.platformio.org/en/latest/userguide/cmd_device.html

But https://docs.platformio.org/en/latest/userguide/cmd_serialports.html?foo=bar gives 404.

[ivankravets]

Are you sure you don't have an extra setup for redirecting from

You can reproduce this bug with any URL. For example, https://docs.readthedocs.io/page/webhooks.html?arg=test

[stsewd]

Sorry, I didn't know about the /page redirect.

[stsewd]

Ok, that's a different problem that the one I found before

https://github.com/rtfd/readthedocs.org/blob/d91807c5c772fbfdb4be5ff928c9268d4be05e80/readthedocs/core/urls/__init__.py#L13-L16

[stsewd]

This is deployed and working!

[ivankravets]

@stsewd thank you so much!!! 😊