fix: redirect url on model download

containers / podman-desktop-extension-ai-lab

Work with LLMs on a local environment using containers

https://podman-desktop.io/extensions/ai-lab

Apache License 2.0

170 stars 31 forks source link

fix: redirect url on model download #1655

Closed axel7083 closed 3 weeks ago

axel7083 commented 3 weeks ago

What does this PR do?

When redirecting through the location header, website may provide only the path (E.g. /hello) when we expect a full url.

Screenshot / video of UI

N/A

What issues does this PR fix or reference?

Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/1651

How to test this PR?

[x] unit tests has been provided

axel7083 commented 3 weeks ago

When the new URL is computed, it is computed from the URL that initiated the workflow but it should be computed from the URL that caused the redirection in case there are several redirections in the flow

Oh nice catch, thanks for noticing that :+1: