Skip to content

Commit

Permalink
fix: set charset in webmanifest content-type and also test serving it
Browse files Browse the repository at this point in the history
  • Loading branch information
open-dynaMIX committed Jan 4, 2021
1 parent c7b897d commit 853209d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 5 additions & 0 deletions tests/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,11 @@ class TestsRequests:
200,
"application/xml; charset=UTF-8",
),
(
"static/favicons/site.webmanifest",
200,
"application/manifest+json; charset=UTF-8",
),
(
"static/fontawesome-free-5.0.2/css/fontawesome-all.min.css",
200,
Expand Down
2 changes: 1 addition & 1 deletion webui.lua
Original file line number Diff line number Diff line change
Expand Up @@ -432,7 +432,7 @@ local function get_content_type(file_type)
elseif file_type == 'mp3' then
return 'audio/mpeg'
elseif file_type == 'webmanifest' then
return 'application/manifest+json'
return 'application/manifest+json; charset=UTF-8'
end
end

Expand Down

0 comments on commit 853209d

Please sign in to comment.