lug/invidious
7
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

API: Fix duplicated query parameters in proxied video URLs (#4587)

Browse Source 
This pull request fixes that bug that was causing the query parameters to get
doubled in the streaming URLs when '?local=true' is passed to the
'/api/v1/videos/{id}' API endpoint.

Before: host/path?parameters?parameters
After: host/path?parameters

No associated open issue
This commit is contained in:
Samantaz Fox 2024-07-21 14:13:54 +02:00
commit 1ff0775f4b
No known key found for this signature in database
GPG Key ID: F42821059186176E
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/invidious/http_server/utils.cr
View File

@ -11,11 +11,12 @@ module Invidious::HttpServer
params = url.query_params params = url.query_params
params["host"] = url.host.not_nil! # Should never be nil, in theory params["host"] = url.host.not_nil! # Should never be nil, in theory
params["region"] = region if !region.nil? params["region"] = region if !region.nil?
url.query_params = params
if absolute if absolute
return "#{HOST_URL}#{url.request_target}?#{params}" return "#{HOST_URL}#{url.request_target}"
else else
return "#{url.request_target}?#{params}" return url.request_target
end end
end end