RemoteImageProvider ignores query string parameters in remote url

[sniffdk]

Which version are you using?

17.0.0

Bug summary

When the remote image provider resolves the remote url for fetching an image, it uses an extension method on the Request.Path property called GetSourceUrlForRemoteImageProviderUrl.
However, this effectively strips any query parameters from the path.
I'm in a situation now where my remote images contains a signature query parameter, that is necessary for getting an image back.

So, when the image provider constructs the source url to use, it should consider both path and query.
It would make sense to add allowed query parameters to the options object.
Eg:

{
  "Prefix": "/our",
  "RemoteUrlPrefix": "https://our.umbraco.org/",
  "AllowedDomains": [ "our.umbraco.org" ],
  "AllowedQueryParameters":  [ "signature", "timestamp" ]
}

Steps to reproduce

Expected result / actual result

No response

[skttl]

How does your urls look like then? /our/some-image.jpg?width=100&height=100&signature=something&timestamp=1234?

What if the width/height parameters is needed too? And if they need to be different from the ones you use yourself. Maybe I'm overthinking it :)

I think maybe we should pass through all querystring parameters?

[sniffdk]

Hi @skttl
Thanks for the quick reply 🙂

My urls could look something like this:
assetstream/31792/50412.jpeg?timestamp=2025-10-15T12%3a29%3a29.6302722&download=False

Would be fine to pass through all querystring params I think, perhaps as an option, to make sure we are not introducing a breaking change ?